<i18n dev> Integrated: 8368001: java/text/Format/NumberFormat/NumberRoundTrip.java timed out
Justin Lu
jlu at openjdk.org
Mon Nov 24 21:25:09 UTC 2025
On Thu, 20 Nov 2025 22:55:41 GMT, Justin Lu <jlu at openjdk.org> wrote:
> JDK-8260555 exposed test/jdk/java/text/Format/NumberFormat/NumberRoundTrip.java as being susceptible to timeouts.
> While the timeout factor is reverted, this test was testing too many inputs, and should be changed.
> Originally, it was testing (~1000 locales * 4 factories) against ~30 numeric inputs. The test does not need to be exhaustive to this degree.
>
> The total amount of locales is changed to sample around a 1/4 of those provided by the provider. The numeric inputs are reduced to about ~20. (The fixed ones remain, and the random ones are cut in half.)
>
> Locally, I observe average execution for this test to be about ~1.3 seconds, compared to ~3.4 seconds previously. The PR also does a drive-by clean up of the test, i.e. getting rid of the unnecessary escaping + string manipulation/printing and separating the test inputs from the actual testing work done.
This pull request has now been integrated.
Changeset: 507a6d32
Author: Justin Lu <jlu at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/507a6d327f1c613a130273727ee4154b5b4d7ca4
Stats: 230 lines in 1 file changed: 60 ins; 132 del; 38 mod
8368001: java/text/Format/NumberFormat/NumberRoundTrip.java timed out
Reviewed-by: naoto
-------------
PR: https://git.openjdk.org/jdk/pull/28443
More information about the i18n-dev
mailing list