adding rsockets support into JDK

Lu, Yingqi at
Fri Nov 16 17:06:38 UTC 2018

Hi Chris and Daniel,

Thank you very much for your timely help! I already generated the Javadoc. Will attach it to the CSR shortly.


From: Chris Hegarty [mailto:chris.hegarty at]
Sent: Friday, November 16, 2018 9:04 AM
To: Lu, Yingqi < at>
Cc: nio-dev at; Viswanathan, Sandhya <sandhya.viswanathan at>; Aundhe, Shirish <shirish.aundhe at>; Kaczmarek, Eric <eric.kaczmarek at>
Subject: Re: adding rsockets support into JDK


On 16 Nov 2018, at 14:29, Lu, Yingqi < at< at>> wrote:

I already updated in the version 17 of the patch with your

The specification in jdk/net/ and
jdk/net/ look good to me.

I have not generated Javadoc before. Would you please let me know what
kind of flags/options I need to use for the two classes?

Similar to what Daniel suggested. Here’s what I did with stubbed out
versions of the two above files from the `.17` webrev:

$ make docs
$ ...
$ mkdir -p rsocket_docs/api
$ cp build/macosx-x64/images/docs/api/stylesheet.css rsocket_docs/api/
$ cp build/macosx-x64/images/docs/api/index.html rsocket_docs/api/
$ cp -r build/macosx-x64/images/docs/api/<> rsocket_docs/api/
$ scp -r rsocket_docs/ chegar at<mailto:chegar at>:

Online docs:
Also, in addition to attach the Javadocs for the two classes, anything else I
need to modify the CSR?

At least an outline of the final classes should be included. No need
to duplicate the spec since you will be attaching a zipped bundle of the
docs. Good to include an online viewable link for convenience, and also
attach a zipped bundled of a minimal doc or specdiff for the archives.



-----Original Message-----
From: Chris Hegarty [mailto:chris.hegarty at]
Sent: Friday, November 16, 2018 3:19 AM
To: Lu, Yingqi < at< at>>
Cc: nio-dev at<mailto:nio-dev at>; Viswanathan, Sandhya
<sandhya.viswanathan at<mailto:sandhya.viswanathan at>>; Aundhe, Shirish
<shirish.aundhe at<mailto:shirish.aundhe at>>; Kaczmarek, Eric <eric.kaczmarek at<mailto:eric.kaczmarek at>>
Subject: Re: adding rsockets support into JDK


On 15/11/18 19:12, Lu, Yingqi wrote:

Hi Chris,

Here is the version 17 of the patch:

This looks much better. Some final spec comments:

1) src/<>

 Looks good.

2) src/<>

a) class-level description now seems a little odd, and heavily weighted
   towards channels, how about something like:

   * Factory methods for creating RDMA-based TCP sockets and channels.
   * <p> The {@link #openSocket() openSocket} and {@link
   * #openServerSocket() openServerSocket} methods create RDMA-based
   * TCP sockets.
   * <p> The {@link #openSelector() openSelector ...

 b) Maybe put the security manager verbiage in its own paragraph, since
    it relates to both sockets and channels.

   * <p> When a security manager is installed ...

 c) Please remove the trailing <p> on line L60.

 d) openSocket: "Creates an unconnected RDMA socket."

   Given that the socket options mention that the option is allowed to
   be "set before the socket is bound or connected", then maybe this
   opening sentence should be:
       "Creates an unbound and unconnected RDMA socket.
   Note: the other factory methods are clear on this point already.

3) Minor: personally I'd remove the @see ...NetworkChannel
   It adds no value.

That's it.

Is it possible to generate the javadoc for these classes? It would be
useful to have to attach to the CSR, or even to be viewable as a link from the

Can you please update the CSR and I will then add myself as reviewer?


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the nio-dev mailing list