[crac] RFR: 8351402: [CRaC] Use System.nanoTime() in TimedWaitingTest [v2]
Timofei Pushkin
tpushkin at openjdk.org
Fri Mar 7 14:41:01 UTC 2025
On Fri, 7 Mar 2025 13:32:47 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:
>> Timofei Pushkin has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix time check
>
> test/jdk/jdk/crac/java/lang/System/TimedWaitingTest.java line 122:
>
>> 120: task.run();
>> 121: long after = System.nanoTime();
>> 122: if (after - before < WAIT_TIME_MILLIS) {
>
> nanoTime reports time in nanoseconds, so this check is no longer valid.
Well, at least we would probably never see the test failing again :)
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/209#discussion_r1985165682
More information about the crac-dev
mailing list