RFR: 8173607: JMX RMI connector should be in its own module
daniel.fuchs at oracle.com
Wed Feb 1 15:29:22 UTC 2017
On 01/02/17 05:11, Mandy Chung wrote:
>> On Jan 31, 2017, at 10:26 AM, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
>> Please find below a patch for:
>> 8173607: JMX RMI connector should be in its own module
> This mostly looks good. I’d like to see an updated webrev after you sync with JDK-8173608. I believe you can revert test/sun/management/jdp and test/sun/management/jmxremote tests change and ConnectorBootstrap.java as you noted. Also you can run jdeps —-check on java.rmi, java.management, and jdk.management.agent to update its requires and qualified exports. java.management should no longer require java.rmi and the qualified exports from java.rmi is not needed.
Here is the updated webrev, rebased after pulling JDK-8173608, and with
your feedback below integrated.
I am pleased to report that java.management no longer requires
java.rmi or java.naming :-)
Compared to previous version, then RMIExporter has been moved
to java.management.rmi, various module-info.java have been
cleaned up, some @modules in tests have been updated (mostly
due to the RMIExporter move).
I have also improved some javadoc comments in JMXConnectorFactory.java
No changes in build files compared to webrev.05
> java.management.rmi module-info.java
> 32 * @see javax.management.remote.rmi
> This @see is not needed. This package is listed in the exported packages table.
> I like the ProviderFinder approach to look up the custom providers and platform providers; and shared code used by both JMXConnectorFactory and JMXConnectorServerFactory which is good.
> Nit: line 481-491 - this is javadoc and probably /* .. */ comment block is more appropriate here.
>> Some further cleanup of java.base and java.rmi module-info.java
>> should become possible once JDK-8173608 is in (as well as moving
>> RMIExporter to java.management.rmi - which is not possible yet).
>> Further cleanup of @modules in tests might become possible as
> Hope there will be a bulk @modules cleanup some time.
More information about the core-libs-dev