RFR: 7903961: Make default timeout configurable

Christian Stein cstein at openjdk.org
Thu Apr 3 10:46:20 UTC 2025


On Tue, 1 Apr 2025 09:29:47 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> src/share/classes/com/sun/javatest/regtest/exec/RegressionScript.java line 1131:
>> 
>>> 1129:     Map<String, String> getTestProperties()  {
>>> 1130:         // initialize the properties with standard properties common to all tests
>>> 1131:         Map<String, String> p = new TreeMap<>(params.getBasicTestProperties());
>> 
>> This change sorts all basic test properties by their name; keeping the insertion order was never a promise.
>
> Does the new proposed feature of configuring a default timeout continue to work without this change to using a `TreeMap`? If yes, then I think it would be good to separate this change out from this PR.

Good point. Reverting this change.

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

PR Review Comment: https://git.openjdk.org/jtreg/pull/253#discussion_r2026723571


More information about the jtreg-dev mailing list