Thanks for the review Roger.

please see the webrev incorporating review comments.

http://cr.openjdk.java.net/~uvangapally/webrev/2017/8181895/webrev.01/

-Ujwal.


On 7/20/2017 2:16 AM, Roger Riggs wrote:
Hi Ujwal,

As Alan noted earlier, it would be helpful in the package javadoc of javax.management.remote.rmi if there was some mention of the class or package within the javadoc that supported the mentioned dynamic classloading. As is, the feature is mentioned but with no suggestion about where to start to find the details.

The other references to the specification are fine.

Roger



On 7/18/2017 7:20 AM, Ujwal Vangapally wrote:
Hi,

kindly review the changes made.

previously technotes were present in pubs repo at /pubs/docs/technotes/guides/

pubs repo  has been removed by JDK-8175825

now we can't include content from that repo in our generated docs.

currently there is no other way to access the content available in technotes .

JMX specification is available from the jcp (https://jcp.org/aboutJava/communityprocess/mrel/jsr160/index2.html) which i used to replace previous link to spec in tech notes.

Sufficient description is already present as part of existing java doc.

Purpose of the link to technotes was to provide more information.

Replaced with alternate links where ever possible.


https://bugs.openjdk.java.net/browse/JDK-8181895

webrev : http://cr.openjdk.java.net/~uvangapally/webrev/2017/8181895/webrev.00/


Thanks,

ujwal.




Reply via email to