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

Igor Ignatyev iignatyev at openjdk.java.net
Wed Apr 14 17:26:38 UTC 2021


On Tue, 30 Mar 2021 08:24:09 GMT, Evgeny Nikitin <enikitin at openjdk.org> wrote:

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

LGTM

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

Marked as reviewed by iignatyev (Reviewer).

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


More information about the hotspot-compiler-dev mailing list