RFR: 8287165: JFR: Add logging to jdk/jfr/api/consumer/recordingstream/TestOnEvent.java

Erik Gahlin egahlin at openjdk.java.net
Mon May 23 16:17:25 UTC 2022


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

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

Commit messages:
 - Initial

Changes: https://git.openjdk.java.net/jdk/pull/8847/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8847&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8287165
  Stats: 9 lines in 2 files changed: 8 ins; 1 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8847.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8847/head:pull/8847

PR: https://git.openjdk.java.net/jdk/pull/8847


More information about the hotspot-jfr-dev mailing list