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