RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1 [v3]
SendaoYan
syan at openjdk.org
Tue Aug 19 06:40:45 UTC 2025
On Tue, 19 Aug 2025 06:04:49 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> DELETED - I confused the timeout with the timeout-factor. New comment below.
>
> No change should be made to any explicit setting of the timeoutFactor in general as that could cause mass timeouts to occur (old default timeout = 120 * 10 = 1200 but new default = 120 * 2.5 = 300!).
>
> However I see the concern of @sendaoYan because individual tests may now get much larger timeout values when run with the non-default timeoutFactor because they have been adjusted for the new default. I don't see any solution to this dilemma.
But what this PR do is change the timeoutFactor in general and find out all the tests which may timeout after the timeoutFactor has been changed.
The old default timeout before this PR is 120 * 4, after this PR the new default is 120 * 1
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26749#discussion_r2284273170
More information about the core-libs-dev
mailing list