badaiaqrandista commented on a change in pull request #8644:
URL: https://github.com/apache/kafka/pull/8644#discussion_r434171589
##########
File path:
clients/src/main/java/org/apache/kafka/clients/CommonClientConfigs.java
##########
@@ -42,9 +42,11 @@
+ "servers (you may
want more than one, though, in case a server is down).";
public static final String CLIENT_DNS_LOOKUP_CONFIG = "client.dns.lookup";
- public static final String CLIENT_DNS_LOOKUP_DOC = "Controls how the
client uses DNS lookups. If set to <code>use_all_dns_ips</code> then, when the
lookup returns multiple IP addresses for a hostname,"
- + " they will all be
attempted to connect to before failing the connection. Applies to both
bootstrap and advertised servers."
- + " If the value is
<code>resolve_canonical_bootstrap_servers_only</code> each entry will be
resolved and expanded into a list of canonical names.";
+ public static final String CLIENT_DNS_LOOKUP_DOC = "Controls how the
client uses DNS lookups."
+ + " If set to
<code>use_all_dns_ips</code>, attempt to connect to all IP addresses returned
by the lookup and use the first one that connects successfully."
+ + " If set to
<code>default</code>, connect to the first IP address returned by the lookup,
even if the lookup returns multiple IP addresses."
+ + " If set to
<code>resolve_canonical_bootstrap_servers_only</code>, each entry will be
resolved and expanded into a list of canonical names."
+ + " Note that
<code>default</code> is deprecated and will be removed in future release.";
Review comment:
Sorry, I do not understand why you are referring to "advertised
servers". Instead, I have changed the explanation to clarify that this only
applies "if bootstrap hostname is an alias to multiple canonical names".
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
