8022206: Intermittent test failures in java/lang/ProcessBuilder/Basic.java

[Alan Bateman]

On 19/11/2013 20:14, Martin Buchholz wrote:
> In jsr166 tests we have mostly switched to 10 second timeouts to mean 
> "forever".
> But in ProcessBuilder tests we are starting up new java processes with 
> their well-known startup problems, so using a much larger value of 
> "forever" seems reasonable.  I vote for 1 minute.  You can write
>
> waitFor(1, TimeUnit.MINUTES);
>
I agree that a timeout on that order might be needed when starting 
process. However I think this one is just starting a thread that 
exercises the new waitFor method.

-Alan.