Thanks Jaroslav and Daniel for the code review.
Still not get review from a code reviewer, not sure I could do push with the current reviews, this is a simple test fix, and only changing way to start a JMX connector server/client: allowing RMI Sever to use an any free port, instead to specify a fixed port.

Thanks,
Shanliang

Daniel Fuchs wrote:
Hi Shanliang,

This change looks good.
There's probably a subtle difference between invoking a command
through a user created JMXConnectorServer as opposed as sending it
through the default agent - but I don't think that's what matters
here.

cheers,

-- daniel

On 6/12/13 9: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





Reply via email to