<i18n dev> RFR: 8368001: java/text/Format/NumberFormat/NumberRoundTrip.java timed out [v2]

Justin Lu jlu at openjdk.org
Fri Nov 21 17:47:43 UTC 2025


> 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.

Justin Lu has updated the pull request incrementally with one additional commit since the last revision:

  Use jdk.test.lib.RandomFactory

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28443/files
  - new: https://git.openjdk.org/jdk/pull/28443/files/bf14ea50..dca988e7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28443&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28443&range=00-01

  Stats: 9 lines in 1 file changed: 6 ins; 1 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/28443.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28443/head:pull/28443

PR: https://git.openjdk.org/jdk/pull/28443


More information about the i18n-dev mailing list