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