On Tue, 25 Aug 2026 16:25:35 GMT, Fabian Meumertzheim <[email protected]> wrote:

>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Fabian Meumertzheim has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Address second round of comments

test/jdk/java/util/concurrent/forkjoin/GetMultipleWaiters.java line 62:

> 60:             } catch (Throwable ignore) {
> 61:             }
> 62:         }, "Get-waiter-B", Thread.State.WAITING);

I wonder if we really need `b` if we launch this test in othervm with a 
timeout. Would you mind trying removing `b` and calling `task.get(…)` on the 
main thread and seeing if that is enough? It would save a chunk of test case 
complexity if it pans out.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32485#discussion_r3861774247

Reply via email to