RFR 9: 8168517 : java/lang/ProcessBuilder/Basic.java failed

[Roger Riggs]

Hi David,

On 10/27/2016 12:57 PM, David Holmes wrote:
> Hi Roger,
>
> On 28/10/2016 1:44 AM, Roger Riggs wrote:
>> Please review a test fix for a timeout on a busy system in
>> Process.waitFor a destroyed process.
>
> Won't that now cause the test to hang until timed-out by the harness?
yes, but an in-app timeout is not much different than the harness timeout
so I didn't complicate the test. Under normal system loading it should 
be 1-4 seconds.

I contemplated increasing the timeout but given the issue is system loading
I didn't have a good idea what to raise it to.  30sec, 1 min, 2 min.
and the harness timeout is 2min.

Roger

>
> David
>
>> Webrev:
>> http://cr.openjdk.java.net/~rriggs/webrev-basic-destroy-8168517/
>>
>> Thanks, Roger
>>