RFR: 8349206: j.u.l.Handler classes create deadlock risk via synchronized publish() method

[Stuart Marks]

On Thu, 6 Feb 2025 12:07:57 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).

A couple days ago the bot warned

> This pull request contains merges that bring in commits not present in the target repository. 

I'm not sure why this happened. It might be because of this commit earlier in this branch:

> [Merge remote-tracking branch 'origin/JDK-8349206-1' into JDK-8349206-1](https://github.com/openjdk/jdk/pull/23491/commits/c35e5198838f1088e137e0e8ad480d44bb303057)

but I'm not sure. I'm also not sure whether or not this will have any ill effects if we proceed.... Not sure what to do about this.

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

PR Comment: https://git.openjdk.org/jdk/pull/23491#issuecomment-2655157065