On Thu, 22 May 2025 14:19:09 GMT, Viktor Klang <vkl...@openjdk.org> wrote:
> Barring other effects, it is likely that this test just needs to have some > longer timeouts to ensure that a stall isn't likely to fail the test case. > The shouldn't have any impact on test execution duration, as the test looks > at tasks which *have not executed yet*. test/jdk/java/util/concurrent/tck/ScheduledExecutorTest.java line 703: > 701: final ScheduledThreadPoolExecutor p = new > ScheduledThreadPoolExecutor(1); > 702: List<ScheduledFuture<?>> tasks = new ArrayList<>(); > 703: final int DELAY = 100; What do you think of trying to change this to `Utils.adjustTimeout(100)` so the env could change this depending on the factor used? This way may be the delay won't need to be as large. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25394#discussion_r2102774011
