RFR 9 8043477: java/lang/ProcessBuilder/Basic.java failed with: java.lang.AssertionError: Some tests failed

roger riggs roger.riggs at oracle.com
Thu Nov 13 15:37:00 UTC 2014

Please re-review,

Corrected the webrev, and a few changes to consistently use the same
units, thanks for the review:

The timeout time is extended to wait up to 7 seconds
and the initial waitFor should last at least 1 second.



On 11/12/2014 10:42 PM, Martin Buchholz wrote:
> On Wed, Nov 12, 2014 at 6:25 PM, roger riggs <roger.riggs at oracle.com> wrote:
>> Hi Martin,
>> I think that sleep is the version that spawns a JavaChild process and
>> has its own comment interpreter. See line 309: Am I getting the sleeps mixed
>> up?
> Ah, good point - yes, you're right.  Although we could always bump up
> the sleep of the JavaChild from 10 if we wanted to - it should be
> comfortably bigger than any individual test case's need.

More information about the core-libs-dev mailing list