RFR: 8308038: java/util/concurrent/ThreadPerTaskExecutor/ThreadPerTaskExecutorTest.java timed out

[Daniel Fuchs]

On Mon, 22 May 2023 10:09:27 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> test/jdk/java/util/concurrent/ThreadPerTaskExecutor/ThreadPerTaskExecutorTest.java line 578:
>> 
>>> 576:             try {
>>> 577:                 scheduleInterruptAt("java.util.concurrent.ThreadPerTaskExecutor.invokeAny");
>>> 578:                 executor.invokeAny(Set.of(task));
>> 
>> Seems strange to test invokeAny when there is only one task?
>
>> Seems strange to test invokeAny when there is only one task?
> 
> There are several tests further up that exercise invokeAny with more than one task. This one is testing interrupt of invokeAny, we could expect it to test interrupting with many tasks if needed.

OK - looks fine to me then.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14072#discussion_r1200349759