On Sun, 21 May 2023 13:52:06 GMT, Alan Bateman <al...@openjdk.org> wrote:
> This is a test only change to the unit test for the ExecutorService returned > by Executors.newThreadPerTaskExecutor. The tests for interrupting invokeAll > assume the threads started to execute the tasks do actually execute the task > code. The refresh in JEP 444 changed the implementation to use FutureTask, > and FutureTask checks the interrupt status before it executes the task code. > So some intermittent timeouts of the tests for interrupting invokeAll as > those tests were waiting for the task to complete. > > The main change is that the tests for interrupting invokeAll are changed to > interrupt when the main thread blocks in invokeAll. They are also changed to > check if the task started or not. The tests for interrupting invokeAny > already did this, but these are changed to use the same infrastructure to > avoid having two styles of tests in the same source file. This pull request has now been integrated. Changeset: fe8c689e Author: Alan Bateman <al...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/fe8c689eeea4fa19d4a8742f4ef1d8216f1394e6 Stats: 206 lines in 1 file changed: 87 ins; 57 del; 62 mod 8308038: java/util/concurrent/ThreadPerTaskExecutor/ThreadPerTaskExecutorTest.java timed out Reviewed-by: dfuchs, jpai ------------- PR: https://git.openjdk.org/jdk/pull/14072
