RFR: 8262060: compiler/whitebox/BlockingCompilation.java timed out [v2]

Evgeny Nikitin enikitin at openjdk.java.net
Tue Mar 30 08:24:09 UTC 2021


> Please review this small fix for a test.
> 
> Test fails sometimes when run with UsageTracker enabled. For some reason, a loading of ThreadLocalRandom can happen during the test run, and this invalidates Random.nextInt method (because it's not the only implementation now).
> 
> Fixed by pre-loading ThreadLocalRandom. Tested by multiple runs with UsageTracker enabled - approx. 1 out of 20-30 test runs fails without the fix and no failures spotted with the fix.

Evgeny Nikitin has updated the pull request incrementally with one additional commit since the last revision:

  Get rid of dependency on the Random class

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3224/files
  - new: https://git.openjdk.java.net/jdk/pull/3224/files/cd8d1d5e..e09eaed2

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3224&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3224&range=00-01

  Stats: 6 lines in 1 file changed: 0 ins; 5 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3224.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3224/head:pull/3224

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


More information about the hotspot-compiler-dev mailing list