On Tue, 8 Apr 2025 11:03:39 GMT, David Beaumont <d...@openjdk.org> wrote:

>> 8353683: 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).
>
> This is a redo of https://github.com/openjdk/jdk/pull/23491 with a tiny 
> additional edit to remove an unwanted `<p>` element in the public JavaDoc 
> (see 
> https://github.com/openjdk/jdk/pull/24504/commits/772a03e0ca08da9412b85ceb8de5adf4fb52d2ac
>  ).

OK I ran tier1 and tier2 + logging test and got no failures. @david-beaumont 
can you integrate and let me know?

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

PR Comment: https://git.openjdk.org/jdk/pull/24504#issuecomment-2793857309

Reply via email to