Hi Shanliang, the change looks fine. The only small thing I have is whether it is good to wrap cc.close(); cs.stop();
in try/catch and swallow the exception. Wouldn't it be better to let the test fail if something goes wrong with closing JMXConnector and JXMConnectorServer? -JB- On 06/12/2013 09:56 AM, shanliang wrote: > Re-send with right subject :) > > Shanliang > > shanliang wrote: >> Hi, >> >> The fix is to address https://jbs.oracle.com/bugs/browse/JDK-8016221 >> >> Webrev: http://cr.openjdk.java.net/~sjiang/JDK-8016221/webrev.00/ >> >> Instead to use a fixed port to run a JMX connector, we specify the >> port as 0: >> JMXServiceURL url = new JMXServiceURL("rmi", null, 0); >> to allow JMX to select a free port. >> >> and the fix makes sure to close the client and server when the test is >> finished (passed or failed). >> >> Shanliang >> >> >
