On 18/12/2012 06:52, Mandy Chung wrote:

I looked at the source changes that look good to me. I believe RMIConnector.connect() will throw IOException("iiop not supported") if the given JMXServiceURL specifies iiop protocol but the implementation doesn't support iiop. Do you think it's worth clarifying that in RMIConnector spec - the spec for RMIConnector(JMXServiceURL, Map) constructor has service:jmx:iiop://.... listed as a valid URL.

Mandy
I mulled over this when doing the changes and concluded it wasn't strictly required, but I can go either way on this. It would really just be overriding the description of the IOException so that it's clear it gets thrown for this case too. (RMIConnectorServer.start is the other one). Do others have an opinion?

-Alan.

Reply via email to