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

Erik Gahlin egahlin at openjdk.org
Wed Jun 4 13:41:55 UTC 2025


On Wed, 4 Jun 2025 04:15:12 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: a653ff48
Author:    Erik Gahlin <egahlin at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a653ff48933bfd72c7c3d004ccc5bd0d9c1162cb
Stats:     10 lines in 2 files changed: 6 ins; 4 del; 0 mod

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

Reviewed-by: mgronlun

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

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


More information about the hotspot-jfr-dev mailing list