Re: RR, M 7186723: TEST_BUG: Race condition in sun/management/jmxremote/startstop/JMXStartStopTest.sh

2012-08-25 Thread Alan Bateman
On 25/08/2012 21:16, Dmitry Samersoff wrote: : The test sends couple of jcmd commands to the single server process. jcmd do attach, call, detach for each command. So I have to keep server running for some time. Server process with just while(true) sleep(1); would run forever if test doesn't k

Re: RR, M 7186723: TEST_BUG: Race condition in sun/management/jmxremote/startstop/JMXStartStopTest.sh

2012-08-25 Thread Dmitry Samersoff
Alan, On 2012-08-26 00:00, Alan Bateman wrote: > On 25/08/2012 16:18, Dmitry Samersoff wrote: >> Hi Everybody, >> >> Test was changed to better support windows. >> >> http://cr.openjdk.java.net/~dsamersoff/7186723/webrev.01/ >> >> -Dmitry >> > You will also need to remove the test from the exclude

Re: RR, M 7186723: TEST_BUG: Race condition in sun/management/jmxremote/startstop/JMXStartStopTest.sh

2012-08-25 Thread Alan Bateman
On 25/08/2012 16:18, Dmitry Samersoff wrote: Hi Everybody, Test was changed to better support windows. http://cr.openjdk.java.net/~dsamersoff/7186723/webrev.01/ -Dmitry You will also need to remove the test from the exclude (jdk/test/ProblemList.txt), we had to add it because it was causing

RR, M 7186723: TEST_BUG: Race condition in sun/management/jmxremote/startstop/JMXStartStopTest.sh

2012-08-25 Thread Dmitry Samersoff
Hi Everybody, Test was changed to better support windows. http://cr.openjdk.java.net/~dsamersoff/7186723/webrev.01/ -Dmitry -- Dmitry Samersoff Java Hotspot development team, SPB04 * There will come soft rains ...

Re: JEP 158: Unified JVM logging

2012-08-25 Thread Martijn Verburg
I'll only add to this that an option to provide a Human Readable format out of the box would be desirable, but not as a default. On 15 August 2012 09:44, Kirk Pepperdine wrote: > Hi Dmitry, > > Lets start with this. > > Common logging command-line options for all components > Logging is performed

Re: Review Request: 7193339 Prepare system classes be defined by a non-null module loader

2012-08-25 Thread Paul Sandoz
Hi Mandy, --- old/src/share/classes/com/sun/jmx/remote/internal/IIOPHelper.java Thu Aug 23 12:29:01 2012 +++ new/src/share/classes/com/sun/jmx/remote/internal/IIOPHelper.java Thu Aug 23 12:29:00 2012 @@ -52,7 +52,7 @@ AccessController.doPrivileged(new PrivilegedAction() {

Re: [PATCH FOR REVIEW] 7194035: update tests for upcoming changes for jtreg

2012-08-25 Thread Alan Bateman
On 25/08/2012 00:51, Jonathan Gibbons wrote: Currently, jtreg incorrectly confuses the concept of the /directory/ in which the test's class will be written with the /classpath/ used to locate all of the test's classes, including any library classes. It provides env variable TESTCLASSES and syst