RFR: 8349206: j.u.l.Handler classes create deadlock risk via synchronized publish() method [v13]
Daniel Fuchs
dfuchs at openjdk.org
Thu Apr 3 08:46:10 UTC 2025
On Wed, 2 Apr 2025 21:59:32 GMT, David Beaumont <duke at openjdk.org> wrote:
>> 8349206: j.u.l.Handler classes create deadlock risk via synchronized publish() method.
>>
>> 1. Remove synchronization of calls to publish() in Handlers in java.util.logging package.
>> 2. Add explanatory comments to various affected methods.
>> 3. Add a test to ensure deadlocks no longer occur.
>>
>> Note that this change does not address issue in MemoryHandler (see JDK-8349208).
>
> David Beaumont has updated the pull request incrementally with one additional commit since the last revision:
>
> Tweak wording.
LGTM ; Maybe give some time to @stuart-marks to have a look too.
-------------
Marked as reviewed by dfuchs (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/23491#pullrequestreview-2738993076
More information about the core-libs-dev
mailing list