Integrated: 8287165: JFR: Add logging to jdk/jfr/api/consumer/recordingstream/TestOnEvent.java
Erik Gahlin
egahlin at openjdk.java.net
Tue May 24 15:56:58 UTC 2022
On Mon, 23 May 2022 15:55:06 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:
> Could I have review of PR that adds logging to a test that fails intermittently. Drawback of adding logging is that it might prevent race conditions to occur, but keeping the test on problem list prevents other use cases of the onEvent-method to be tested.
>
> Testing: test/jdk/jdk/jfr/api/consumer/recordingstream/TestOnEvent.java
>
> We can revisit the main issue https://bugs.openjdk.java.net/browse/JDK-8255404 in 6-12 months if hasn't failed.
>
> Thanks
> Erik
This pull request has now been integrated.
Changeset: d888c80b
Author: Erik Gahlin <egahlin at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/d888c80bbad5bf388cdd65ea4d5749f04bbda94b
Stats: 9 lines in 2 files changed: 8 ins; 1 del; 0 mod
8287165: JFR: Add logging to jdk/jfr/api/consumer/recordingstream/TestOnEvent.java
Reviewed-by: mgronlun
-------------
PR: https://git.openjdk.java.net/jdk/pull/8847
More information about the hotspot-jfr-dev
mailing list