Re: RFR: JDK-8131784 : Add tracing info to LowMemoryTest.java to help 8130339 diagnosis

2015-07-30 Thread serguei.spit...@oracle.com
Hi Olivier, I agree with the comments from Jaroslav. A couple of minor comments. +if ( exitCode != 0 ) { Unneeded extra spaces +} catch (InterruptedException e) { +Thread.currentThread().interrupt(); +throw new RuntimeException("Parent process i

Re: RFR: JDK-8131784 : Add tracing info to LowMemoryTest.java to help 8130339 diagnosis

2015-07-30 Thread Jaroslav Bachorik
Hi Olivier, On 30.7.2015 18:55, olivier.lagn...@oracle.com wrote: Hi, Could you please review this fix: bug: https://bugs.openjdk.java.net/browse/JDK-8131784 webrev: http://cr.openjdk.java.net/~olagneau/8131784/webrev.00/ We add tracing information to help diagnosing master bug problem (https:

RFR: JDK-8131784 : Add tracing info to LowMemoryTest.java to help 8130339 diagnosis

2015-07-30 Thread olivier.lagn...@oracle.com
Hi, Could you please review this fix: bug: https://bugs.openjdk.java.net/browse/JDK-8131784 webrev: http://cr.openjdk.java.net/~olagneau/8131784/webrev.00/ We add tracing information to help diagnosing master bug problem (https://bugs.openjdk.java.net/browse/JDK-8130339), and we will monitor th