Re: RFR:8194298 Add support for per Socket configuration of TCP keepalive

[Alan Bateman]

On 14/05/2018 16:30, vyom tewari wrote:

Hi All,

Please find the latest 
webrev(http://cr.openjdk.java.net/~vtewari/8194298/webrev0.5/index.html). 
Only change with the previous wrev(04) is i removed "socket type" as 
Alan suggested and used the default  constructor (Set<SocketOption<?>> 
options = new HashSet<>();) in ExtendedSocketOptions.java


This looks good to me. Not strictly required, but we adjust the wording 
for TCK_QUICKACK to remove "socket type" too.

-Alan