RFR(XS): 8132094: Mark intermittently failuring core-svc tests

2015-07-22 Thread Yekaterina Kantserova
Hi, Could I please have a review of this fix. bug: https://bugs.openjdk.java.net/browse/JDK-8132094 webrev: http://cr.openjdk.java.net/~ykantser/8132094/webrev.00 Thanks, Katja

Re: RFR(XS): 8132094: Mark intermittently failuring core-svc tests

2015-07-23 Thread Jaroslav Bachorik
Looks good! -JB- On 22.7.2015 15:50, Yekaterina Kantserova wrote: Hi, Could I please have a review of this fix. bug: https://bugs.openjdk.java.net/browse/JDK-8132094 webrev: http://cr.openjdk.java.net/~ykantser/8132094/webrev.00 Thanks, Katja

Re: RFR(XS): 8132094: Mark intermittently failuring core-svc tests

2015-07-23 Thread serguei.spit...@oracle.com
Looks good. Thanks, Serguei On 7/22/15 6:50 AM, Yekaterina Kantserova wrote: Hi, Could I please have a review of this fix. bug: https://bugs.openjdk.java.net/browse/JDK-8132094 webrev: http://cr.openjdk.java.net/~ykantser/8132094/webrev.00 Thanks, Katja

Re: RFR(XS): 8132094: Mark intermittently failuring core-svc tests

2015-07-23 Thread Yekaterina Kantserova
Jaroslav, Serguei, thanks for the review! // Katja On 07/23/2015 12:09 PM, serguei.spit...@oracle.com wrote: Looks good. Thanks, Serguei On 7/22/15 6:50 AM, Yekaterina Kantserova wrote: Hi, Could I please have a review of this fix. bug: https://bugs.openjdk.java.net/browse/JDK-8132094 we

Re: RFR(XS): 8132094: Mark intermittently failuring core-svc tests

2015-07-23 Thread Daniel D. Daugherty
What testing of these changes was done? Dan On 7/22/15 7:50 AM, Yekaterina Kantserova wrote: Hi, Could I please have a review of this fix. bug: https://bugs.openjdk.java.net/browse/JDK-8132094 webrev: http://cr.openjdk.java.net/~ykantser/8132094/webrev.00 Thanks, Katja

Re: RFR(XS): 8132094: Mark intermittently failuring core-svc tests

2015-07-23 Thread Yekaterina Kantserova
Hi, In order to test the fix I ran the svc_tools, jdk_management, jdk_jmx and jdk_jdi tests on all of the core platforms. // Katja On 07/23/2015 03:35 PM, Daniel D. Daugherty wrote: What testing of these changes was done? Dan On 7/22/15 7:50 AM, Yekaterina Kantserova wrote: Hi, Could