RFR 8220684 : Process.waitFor(long, TimeUnit) can return false for a process that exited within the timeout
ivan.gerasimov at oracle.com
Thu Mar 14 19:49:55 UTC 2019
The default implementation of Process.waitFor(long, TimeUnit) does not
check if the process has exited after the last portion of the timeout
JDK has two implementations of Process (for Unix and Windows) and they
both override waitFor(), so it's not an issue for them.
Still, it is better to provide a more accurate default implementation.
I'm not quite certain the regression test needs to be included in the
fix. The test does demonstrate the issue with the unfixed JDK and
passed Okay on all tested platforms in Mach5. Yet, I suspect the test
can still show false negative results, as there are no guaranties that
even such simple application as `true` will finish in 100 ms.
I can tag the test as @ignored with a comment, or simply remove it from
Thanks in advance for reviewing!
With kind regards,
More information about the core-libs-dev