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.
