1) Thanks for the update, the KIP now says `
socket.connections.setup.timeout.ms*`*, which sounds good.
2) Not sure 10s is a good default because it unnecessarily times out
connections, only to attempt re-connecting to the same broker (except in
the leastLoadedNode case where it would be useful to have a lower timeout).
Couldn't we just use 30s (request timeout) as the default value, retaining
the current behaviour? Or alternatively, only apply the timeout where it
makes sense to have a lower timeout (i.e. a timeout for leastLoadedNode)?
On Thu, May 14, 2020 at 5:00 AM Cheng Tan <[EMAIL PROTECTED]> wrote: