Integrated: 8295651: JFR: 'jfr scrub' should summarize what was removed

Erik Gahlin egahlin at openjdk.org
Wed Apr 16 20:14:54 UTC 2025


On Tue, 15 Apr 2025 22:22:17 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> Could I have a review of an enhancement that summarizes the results of `jfr scrub`?
> 
> To accommodate this feature, I made some additional changes:
> 
> - Added a LongMap::values() method.
> - Added a Command::printf(...) method, which is now also used by `jfr summary`.
> - Improved TestScrub.java so it now creates a file that spans multiple chunks, including an empty one.
> 
> To make it easier to locate the scrubbed file, I also changed so the real path instead of the of the absolute path is displayed in the summary 
> 
> See issue for example output.
> 
> Testing: jdk/jdk/jfr
> 
> Thanks
> Erik

This pull request has now been integrated.

Changeset: a2d7f441
Author:    Erik Gahlin <egahlin at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a2d7f441b8616e0cd8c231f3358b1b508e76686d
Stats:     138 lines in 9 files changed: 123 ins; 2 del; 13 mod

8295651: JFR: 'jfr scrub' should summarize what was removed

Reviewed-by: mgronlun

-------------

PR: https://git.openjdk.org/jdk/pull/24669


More information about the hotspot-jfr-dev mailing list