Re: RFR(s): 8228580: DnsClient TCP socket timeout

[Alan Bateman]

On 04/09/2019 18:52, Pavel Rappo wrote:
You are right, it definitely is a blocking operation. I missed it. I was 
focused on
     712         sock.setSoTimeout(timeoutLeft);

So I'd suggest we use explicit connect with timeout

     java.net.Socket#connect(java.net.SocketAddress, int)

Are you okay with that?

Decomposing timeouts is hard to get right. In this case I think you want 
to specify the initial timeout to connect, then set timeoutLeft to be 
the remaining (will be less than "timeout").

-Alan