RFR: 8358536: jdk/jfr/api/consumer/TestRecordingFileWrite.java times out

Erik Gahlin egahlin at openjdk.org
Wed Jun 4 13:19:49 UTC 2025


Could I have a review of the PR that fixes a regression caused by [JDK-8356698](https://bugs.openjdk.org/browse/JDK-8356698)?

The problem is that PrettyWriter is also used by RecordedObject::toString, and by allocating the priority queue when PrettyWriter is instantiated, the memory pressure increases enormously. The fix is to allocate them lazily. 

Testing: 10 * jdk/jdk/jfr/ on Linux, MacOS and Windows.

Thanks
Erik

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

Commit messages:
 - Initial

Changes: https://git.openjdk.org/jdk/pull/25633/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25633&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8358536
  Stats: 10 lines in 2 files changed: 6 ins; 4 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/25633.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25633/head:pull/25633

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


More information about the hotspot-jfr-dev mailing list