On Mon, 9 Feb 2026 10:35:04 GMT, Daniel Fuchs <[email protected]> wrote:
> `java/util/logging/ParentLoggersTest.java` has been observed failing > intermittently - nut rarely - in the loom repo. > After inspection of the code Logger::doSetParent was found to do multiple > read of the volatile `parent` field. After changing the method to do a single > read the test has no longer been observed failing. Unfortunately the logs for the failures have been lost - but IIRC the issue was that the list of expected loggers did not match the list of returned loggers. At the time the initial loggers were not printed - so I had a suspicion that the issue might have laid there. In any case, I believe the change I am proposing is a good change: the `parent` field should only be read once. The additional logging in the test may help if the test fails again, so I would like to push this change, even if the issue was effectively elsewhere. ------------- PR Comment: https://git.openjdk.org/jdk/pull/29634#issuecomment-3876910746
