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