RFR: 8261279: sun/util/resources/cldr/TimeZoneNamesTest.java timed out

[Naoto Sato]

On Mon, 8 Feb 2021 23:42:04 GMT, Joe Wang <joehw at openjdk.org> wrote:

>> Marked as reviewed by lancea (Reviewer).
>
> The fix looks ok to me. Just an interesting note that this test took such a long time to run. Given the amount of locales, limit(30) would reduce the time to approximately 1/5 of the time, that would still be about 100,000ms according to the bug report. That feels pretty long for a single test case that, outside of the test environment, takes only 1s (my local test took 5s without limit(30)). 
> 
> Also, there are a bunch of other tests that also run through all of the available locales, I wonder if they also take a long time to run.

As of JDK15, there are more than 800 locales, so reducing the samples down to 30 makes it less than 4% of the original (~20,000ms on that machine). But I agree that the test took too long than my environment too. Could be a very slow machine.
As to other tests calling available locales,  the reason this test took a long time is that the loop multiplies with the number of time zones which is also huge (600+), so it would be less likely that other locale tests cause timeouts.

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

PR: https://git.openjdk.java.net/jdk/pull/2465