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