<i18n dev> RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1 [v5]
Leo Korinth
lkorinth at openjdk.org
Mon Aug 25 10:52:09 UTC 2025
On Fri, 22 Aug 2025 15:44:28 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
>> Leo Korinth has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update testing.md, remove makefile link, fix bad text
>
> test/hotspot/jtreg/runtime/cds/appcds/LotsOfSyntheticClasses.java line 40:
>
>> 38: * @requires vm.cds
>> 39: * @library /test/lib /test/hotspot/jtreg/runtime/cds/appcds
>> 40: * @run driver/timeout=8000 LotsOfSyntheticClasses
>
> I was expecting `500 * 4 = 2000`, instead of `8000` here.
This is another instance of ([double after quad ](https://github.com/openjdk/jdk/pull/26749#discussion_r2297765975)).
> test/jdk/java/util/HashMap/WhiteBoxResizeTest.java line 60:
>
>> 58: * @comment skip running this test on 32 bit VM
>> 59: * @requires vm.bits == "64"
>> 60: * @run testng/othervm/timeout=960 -Xmx2g WhiteBoxResizeTest
>
> Why not `480`?
This is another instance of ([double after quad ](https://github.com/openjdk/jdk/pull/26749#discussion_r2297765975)).
> test/jdk/java/util/PluggableLocale/LocaleNameProviderTest.java line 34:
>
>> 32: * @build com.foobar.Utils
>> 33: * com.bar.*
>> 34: * @run junit/othervm/timeout=960 -Djava.locale.providers=CLDR,SPI LocaleNameProviderTest
>
> Why not `480`?
This is another instance of ([double after quad ](https://github.com/openjdk/jdk/pull/26749#discussion_r2297765975)).
> test/jdk/jdk/jfr/event/oldobject/TestObjectDescription.java line 49:
>
>> 47: * @library /test/lib /test/jdk
>> 48: * @modules jdk.jfr/jdk.jfr.internal.test
>> 49: * @run main/othervm/timeout=960 -XX:TLABSize=2k jdk.jfr.event.oldobject.TestObjectDescription
>
> Why not `480`?
This is another instance of ([double after quad ](https://github.com/openjdk/jdk/pull/26749#discussion_r2297765975)).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26749#discussion_r2297768577
PR Review Comment: https://git.openjdk.org/jdk/pull/26749#discussion_r2297773391
PR Review Comment: https://git.openjdk.org/jdk/pull/26749#discussion_r2297773648
PR Review Comment: https://git.openjdk.org/jdk/pull/26749#discussion_r2297774209
More information about the i18n-dev
mailing list