RFR: 8372324: jdk/jfr/event/gc/detailed/TestStringDeduplicationEvent.java#Parallel failed

Tue Nov 25 17:34:34 UTC 2025

On Sat, 22 Nov 2025 12:49:44 GMT, Francesco Andreuzzi <fandreuzzi at openjdk.org> wrote:

> The assertion is used to validate a precondition for the test. As long as the deduplication happens inside the `RecordingStream` scope, a `StringDeduplication ` event will be recorded. Thus the assertion is not needed and can be removed.

Looks like getValue() is never used. Can it be removed, and the reflection code? Do we really need the fields to be volatile?

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

PR Comment: https://git.openjdk.org/jdk/pull/28467#issuecomment-3576632211