RFR: 7904105: intermittent test failure in CleanupDirTests.agentvm.conc
Christian Stein
cstein at openjdk.org
Tue Dec 2 14:12:17 UTC 2025
On Tue, 28 Oct 2025 03:12:34 GMT, Vladimir Petko <vpetko at openjdk.org> wrote:
> jtharness does not synchronize calls to Observer. Synchronized keyword needs to be added when processing shared state.
This change looks good-enough for the time being.
As @jaikiran said, there's potential to address this in a more fundamental way, but adding those three `synchronized` modifiers a) doesn't hurt, b) seems to address the flaky test issue, and c) doesn't prevent future improvements.
-------------
Marked as reviewed by cstein (Reviewer).
PR Review: https://git.openjdk.org/jtreg/pull/297#pullrequestreview-3530334046
More information about the jtreg-dev
mailing list