On Thu, 6 Feb 2025 12:07:57 GMT, David Beaumont <d...@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

Reply via email to