Re: Request for review: 7191777: test/java/lang/ProcessBuilder/Basic.java failing intermittently due to additions for 4244896

2012-08-17 Thread Alan Bateman
On 16/08/2012 21:53, David Holmes wrote: On 17/08/2012 2:33 AM, Rob McKenna wrote: Sounds good: http://cr.openjdk.java.net/~robm/7191777/webrev.02/test/java/lang/ProcessBuilder/Basic.java.cdiff.html

Re: Request for review: 7191777: test/java/lang/ProcessBuilder/Basic.java failing intermittently due to additions for 4244896

2012-08-16 Thread David Holmes
On 17/08/2012 2:33 AM, Rob McKenna wrote: Sounds good: http://cr.openjdk.java.net/~robm/7191777/webrev.02/test/java/lang/ProcessBuilder/Basic.java.cdiff.html I think we would want the non

Re: Request for review: 7191777: test/java/lang/ProcessBuilder/Basic.java failing intermittently due to additions for 4244896

2012-08-16 Thread Rob McKenna
Sounds good: http://cr.openjdk.java.net/~robm/7191777/webrev.02/test/java/lang/ProcessBuilder/Basic.java.cdiff.html -Rob On 16/08/12 08:19, Alan Bateman wrote: On 16/08/2012 03:18,

Re: Request for review: 7191777: test/java/lang/ProcessBuilder/Basic.java failing intermittently due to additions for 4244896

2012-08-16 Thread Alan Bateman
On 16/08/2012 03:18, David Holmes wrote: Hi Rob, On 16/08/2012 9:09 AM, Rob McKenna wrote: Hi folks, One of the tests from 4244896 failed once during nightly testing. It isn't known how much of a delay will be necessary in order for it to pass. In any case the tolerance can't really be loosene

Re: Request for review: 7191777: test/java/lang/ProcessBuilder/Basic.java failing intermittently due to additions for 4244896

2012-08-15 Thread David Holmes
Hi Rob, On 16/08/2012 9:09 AM, Rob McKenna wrote: Hi folks, One of the tests from 4244896 failed once during nightly testing. It isn't known how much of a delay will be necessary in order for it to pass. In any case the tolerance can't really be loosened much more without making the test meanin

Request for review: 7191777: test/java/lang/ProcessBuilder/Basic.java failing intermittently due to additions for 4244896

2012-08-15 Thread Rob McKenna
Hi folks, One of the tests from 4244896 failed once during nightly testing. It isn't known how much of a delay will be necessary in order for it to pass. In any case the tolerance can't really be loosened much more without making the test meaningless so I've decided to remove it. http://cr.o