[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: RFR: JDK-8181895 javax management docs contain links to technotes
From: Roger Riggs <Roger.Riggs () Oracle ! com>
Date: 2017-07-24 18:44:04
Message-ID: c76ed61e-6d11-1849-fe17-95ca9395bb78 () Oracle ! com
[Download RAW message or body]
Hi Ujwal,
The updated links look fine.
Thanks, Roger
On 7/24/2017 1:48 PM, Ujwal Vangapally wrote:
> 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.
>>>
>>>
>>
>
[Attachment #3 (text/html)]
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFE0">
Hi Ujwal,<br>
<br>
The updated links look fine.<br>
<br>
Thanks, Roger<br>
<br>
<br>
<div class="moz-cite-prefix">On 7/24/2017 1:48 PM, Ujwal Vangapally
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:b51ce470-1822-828b-edd5-c722c3ed049b@oracle.com">Thanks
for the review Roger.
<br>
<br>
please see the webrev incorporating review comments.
<br>
<br>
<a class="moz-txt-link-freetext" \
href="http://cr.openjdk.java.net/~uvangapally/webrev/2017/8181895/webrev.01/">http://cr.openjdk.java.net/~uvangapally/webrev/2017/8181895/webrev.01/</a>
<br>
<br>
-Ujwal.
<br>
<br>
<br>
On 7/20/2017 2:16 AM, Roger Riggs wrote:
<br>
<blockquote type="cite">Hi Ujwal,
<br>
<br>
As Alan noted earlier, it would be helpful in the package
javadoc of javax.management.remote.rmi
<br>
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.
<br>
<br>
The other references to the specification are fine.
<br>
<br>
Roger
<br>
<br>
<br>
<br>
On 7/18/2017 7:20 AM, Ujwal Vangapally wrote:
<br>
<blockquote type="cite">Hi,
<br>
<br>
kindly review the changes made.
<br>
<br>
previously technotes were present in pubs repo at
/pubs/docs/technotes/guides/
<br>
<br>
pubs repo has been removed by JDK-8175825
<br>
<br>
now we can't include content from that repo in our generated
docs.
<br>
<br>
currently there is no other way to access the content
available in technotes .
<br>
<br>
JMX specification is available from the jcp
(<a class="moz-txt-link-freetext" \
href="https://jcp.org/aboutJava/communityprocess/mrel/jsr160/index2.html">https://jcp.org/aboutJava/communityprocess/mrel/jsr160/index2.html</a>)
which i used to replace previous link to spec in tech notes.
<br>
<br>
Sufficient description is already present as part of existing
java doc.
<br>
<br>
Purpose of the link to technotes was to provide more
information.
<br>
<br>
Replaced with alternate links where ever possible.
<br>
<br>
<br>
<a class="moz-txt-link-freetext" \
href="https://bugs.openjdk.java.net/browse/JDK-8181895">https://bugs.openjdk.java.net/browse/JDK-8181895</a>
<br>
<br>
webrev :
<a class="moz-txt-link-freetext" \
href="http://cr.openjdk.java.net/~uvangapally/webrev/2017/8181895/webrev.00/">http://cr.openjdk.java.net/~uvangapally/webrev/2017/8181895/webrev.00/</a>
<br>
<br>
<br>
Thanks,
<br>
<br>
ujwal.
<br>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic