RFR: 8356171: Increase timeout for testcases as preparation for change of default timeout factor
Leo Korinth
lkorinth at openjdk.org
Fri May 9 09:02:55 UTC 2025
On Thu, 8 May 2025 20:00:21 GMT, Phil Race <prr at openjdk.org> wrote:
> test/jdk/java/awt/font/NumericShaper/MTTest.java
>
> * * @run main/timeout=300/othervm MTTest
>
>
> * * @run main/timeout=1200/othervm MTTest
>
>
> I'm puzzling over why you saw this test fail with timeout = 300 .. or perhaps you saw it fail with 0.7 ? Which would amount to 210 seconds .. that might just be enough to cause it to fail because if you look at the whole test you'll see it wants the core loops of the test to run for 180 seconds.
>
> https://openjdk.github.io/cr/?repo=jdk&pr=25122&range=00#new-144-test/jdk/java/awt/font/NumericShaper/MTTest.java
>
> So 300 was fine, and 1200 isn't needed.
I started with a timeout factor less than `0.7` but I got hindered by CODETOOLS-7903937. That is probably the reason. Maybe I should change the timeout to 400? I think it is reasonable to handle a timeout factor somewhat less than 1 to weed out tight test cases. But maybe 300 is good enough?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25122#issuecomment-2865742871
More information about the compiler-dev
mailing list