RFR: CODETOOLS-7903560: JMH: async profiler reports only the last fork result

Jonas Konrad duke at openjdk.org
Wed Sep 27 11:49:59 UTC 2023


Without this change, async-profiler would write the same result in each fork, overwriting the previous result.

This patch introduces a new "FileResult" class that collects the output paths of each benchmark. In `afterTrial`, the output files are moved to a new path that includes the pid (fork number would be an alternative but it's not as easily available).

At the moment, results of different runs are not coalesced. But with these changes, it would be easy to coalesce at least the results of the "collapsed" output mode. I plan to do this in another PR.

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

Commit messages:
 - Merge branch 'master' into async-fork
 - license headers
 - Include PID in async-profiler file name when forking

Changes: https://git.openjdk.org/jmh/pull/121/files
 Webrev: https://webrevs.openjdk.org/?repo=jmh&pr=121&range=00
  Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903560
  Stats: 333 lines in 3 files changed: 306 ins; 12 del; 15 mod
  Patch: https://git.openjdk.org/jmh/pull/121.diff
  Fetch: git fetch https://git.openjdk.org/jmh.git pull/121/head:pull/121

PR: https://git.openjdk.org/jmh/pull/121


More information about the jmh-dev mailing list