On Mon, 20 Oct 2025 18:00:26 GMT, Daniel Fuchs <[email protected]> wrote:
>> it's shouldAwaitActiveExchange the test that fails
>>
>> final Duration delayDuration = Duration.ofSeconds(Utils.adjustTimeout(20));
>> log("Shutdown triggered with the delay of " +
>> delayDuration.getSeconds());
>> final long elapsed = timeShutdown(delayDuration, startTime);
>> log("Shutdown complete");
>>
>> the shutdown delay has been increased to 20 seconds
>>
>> you are verifying that the server will stop after a max time of 20 seconds +
>> (whatever time its takes to terminate whatever needs. to be terminated)
>>
>> The 1 seconds delay is to activate the Exchange and set it running
>
> The test that fails and has been updated with:
>
>
> final Duration delayDuration = Duration.ofSeconds(Utils.adjustTimeout(20));
>
>
> is `shouldAwaitActiveExchange()` so I guess your comment was intended for
> `shouldAwaitAfterExchange()`.
>
> BTW @myankelev: in `shouldCompeteAfterDelay` is `compete` a typo?
apologies I got my cut and paste all wrong
shouldAwaitActiveExchange
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27670#discussion_r2445747271
