RFR:8194298 Add support for per Socket configuration of TCP keepalive
vyom tewari
vyom.tewari at oracle.com
Sun Apr 15 13:17:46 UTC 2018
On Sunday 15 April 2018 01:33 PM, Alan Bateman wrote:
> On 14/04/2018 08:09, Alan Bateman wrote:
>> :
>>
>> Can you update SocketChannel/SocketOptionTests.java to ensure that
>> SocketChannel is test? We also need to ensure that the new options
>> don't show up in the supportedOptions returned by the channels that
>> don't support these new options.
> Just on this point, I think this needs work in ExtendedSocketOptions
> so that the extended options are organized by socket type (STREAM or
> DGRAM). This will become a lot more obvious once you add tests for
> SocketChannel as its implementation will need a change to pick up the
> extended options for STREAM sockets. It will also avoid the filtering
> in PlainDatagramSocketImpl that you've added to work around the issue
> there.
Even i thought the same, when i added TCP_QUICKACK but somehow i choose
not to do at that time. I will modify the tests and will do the other
changes suggested by you.
Vyom
>
> -Alan
More information about the net-dev
mailing list