frankgh commented on code in PR #227:
URL: https://github.com/apache/cassandra-sidecar/pull/227#discussion_r2157326559
##########
server/src/main/java/org/apache/cassandra/sidecar/config/ServiceConfiguration.java:
##########
@@ -179,4 +180,11 @@ default WorkerPoolConfiguration
serverInternalWorkerPoolConfiguration()
* @return the configuration relevant to the coordination functionality of
Sidecar
*/
CoordinationConfiguration coordinationConfiguration();
+
+ /**
+ * @return Sidecar's DNS Resolver to use. Default will be to provide
forward and reverse DNS resolution which
+ * provides both the hostname & address. The alternative, resolveToIp,
only provides forward resolution meaning
+ * the IP address will be returned for both hostname & address.
+ */
+ DnsResolver.DnsResolverEnum dnsResolver();
Review Comment:
Here we should use the interface instead.
--
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.
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
