RFR: 8272600: (test) Use native "sleep" in Basic.java [v4]
David Holmes
dholmes at openjdk.java.net
Wed Sep 15 22:58:52 UTC 2021
On Wed, 15 Sep 2021 22:36:14 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> test/jdk/java/lang/ProcessBuilder/Basic.java line 2452:
>>
>>> 2450:
>>> 2451: if (p.waitFor(10, TimeUnit.MILLISECONDS)) {
>>> 2452: System.out.println("WaitFor didn't wait long enough: " + (System.nanoTime() - start));
>>
>> Either the condition or the message seems wrong here. If waitFor returns true then the process has exited and we obviously did wait long enough.
>
> This code is diagnostic.
> After switching to native sleep, I had intermittent failures claiming it did not sleep long enough.
> I was unable to find a specific cause for those failures.
> Many of the tests fail to check if the sleep processes terminate prematurely and if the executable is not found, it never launched.
Okay but my comment still stands.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5239
More information about the core-libs-dev
mailing list