DefaultDnsClient reduce log level for TTL > max (#2557)

Motivation: DefaultDnsClient logs at info level if the TTL from DNS is larger than the maximum configured TTL value. This level of logging can be noisy in production and not necessarily be within the client's control.
apple · May 2, 2023 · 825f235 · 825f235
1 parent fac100a
commit 825f235
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/...ns-discovery-netty/src/main/java/io/servicetalk/dns/discovery/netty/DefaultDnsClient.java b/...ns-discovery-netty/src/main/java/io/servicetalk/dns/discovery/netty/DefaultDnsClient.java
@@ -813,9 +813,11 @@ private void handleResolveDone0(final Future<DnsAnswer<T>> addressFuture,
                     final List<T> addresses = dnsAnswer.answer();
                     ttlNanos = dnsAnswer.ttlNanos();
                     if (ttlNanos > maxTTLNanos) {
-                        LOGGER.info("{} result for {} has a high TTL={}s which is larger than configured maxTTL={}s.",
-                                DefaultDnsClient.this, AbstractDnsPublisher.this,
-                                NANOSECONDS.toSeconds(ttlNanos), NANOSECONDS.toSeconds(maxTTLNanos));
+                        if (LOGGER.isDebugEnabled()) {
+                            LOGGER.debug("{} result for {} has TTL={} > maxTTL={}",
+                                    DefaultDnsClient.this, AbstractDnsPublisher.this, NANOSECONDS.toSeconds(ttlNanos),
+                                    NANOSECONDS.toSeconds(maxTTLNanos));
+                        }
                         ttlNanos = maxTTLNanos;
                     }
                     final List<ServiceDiscovererEvent<T>> events = calculateDifference(activeAddresses, addresses,