On Tue, 23 Sep 2025 13:34:02 GMT, Daniel Fuchs <[email protected]> wrote:

>> we probably could. This test should work equally well with a `send` instead 
>> of `sendAsync` too. But the proposed change is good enough for now.
>
> @sendaoYan is this test run with a timeout factor > 1 in the configuration 
> where it fails?
> If so a possibility could also be to use `Utils.adjustTimeout` to adjust the 
> timeout with the timeout factor.

I see from the log that timeoutFactor is 2.0 - so if the test passes with a 
timeout of 20 instead of 10, using `Utils.adjustTimeout` might be preferable as 
it will extend the timeout on those configurations where timeoutFactor is even 
greater.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27447#discussion_r2372378492

Reply via email to