RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete
Joe Darcy
darcy at openjdk.org
Tue Dec 5 03:49:31 UTC 2023
On Mon, 4 Dec 2023 21:30:44 GMT, Tom Rodriguez <never at openjdk.org> wrote:
> I guess that sort of makes sense. The default timeout factor is 4 I think so I guess I descale the existing total wait of 1000 ms so that the total wait is now Utils.adjustTimeout(250)? Maybe others might offer an opinion? Since these waits are a fixed timeout with no early exit I don't think we want them to be too long but anything in 1-5 seconds really doesn't matter. That's part of why I was just picking a static value.
Looking at running the test suite as a whole, especially for tier 1 tests which are run very frequently, a few seconds here, a few seconds there, over a few thousand tests starts to add up in terms of increasing the minimum time needed to run the sets of tests.
(Separate from this issue, I think it would be a useful exercise to more systematically examine the constant sleeps in the test suite and tune down the constant factor to let the timeout factor do more of the heavy lifting on avoiding spurious failures.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16956#discussion_r1414836607
More information about the core-libs-dev
mailing list