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.

This change has been in the loom repo since Oct 2025 and no sightings of the 
test failure since then. So I think changing it to read parent once is good.

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

Marked as reviewed by alanb (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29634#pullrequestreview-3772642255

Reply via email to