Subject: [VOTE] KIP-602: Change default value for client.dns.lookup


Hi Rajini,

Good point, `getHostString()` actually returns the original host. The
InetSocketAddress code is a bit confusing as it does:

        String host = null;
        try {
            addr = InetAddress.getByName(hostname);
        } catch(UnknownHostException e) {
            host = hostname;
        }
        holder = new InetSocketAddressHolder(host, addr, checkPort(port));

However, the the holder keeps track of the original address too. So, the
change proposed in the KIP should fix the bootstrap behavior too. I still
think the code would be easier to reason about if we returned a List<Node>
instead of List<InetSocketAddress>. But that change can be done
independently of the KIP.

Ismael

On Tue, May 26, 2020 at 1:26 AM Rajini Sivaram <[EMAIL PROTECTED]>
wrote: