RFR: 8029525 - java/lang/ProcessBuilder/Basic.java fails intermittently

Martin Buchholz martinrb at google.com
Thu Dec 5 17:44:02 UTC 2013


sorry for leaving so many little problems in this test.

(In my defense, ProcessBuilder is hard to test in a non-flaky way.)

I think we can come up with a cleaner fix.  I'll post a patch later.


On Thu, Dec 5, 2013 at 6:19 AM, Rob McKenna <rob.mckenna at oracle.com> wrote:

> This failure cropped up again and Roger Riggs spotted that I was looking
> at it from completely the wrong direction. He contributed the following fix:
>
> http://cr.openjdk.java.net/~robm/8029525/webrev.01/
>
> This is to avoid a race between:
>
> thread.interrupt();
> p.destroy();
>
> Hoping to get this reviewed and pushed as soon as possible!
>
> Thanks,
>
>     -Rob
>
>



More information about the core-libs-dev mailing list