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