Merge pull request #1279 from spotify/dxia/log-master-resolution

Add logging for resolution of helios SRV records

Author: davidxia

helios-client/src/main/java/com/spotify/helios/common/Resolver.java

@@ -31,13 +31,17 @@
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.List;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * A utility class that resolves DNS SRV records. Used by {@link PeriodicResolver} to discover
  * helios masters.
  */
 class Resolver {
 
+  private static final Logger log = LoggerFactory.getLogger(Resolver.class);
+
   private static final String HTTPS_SRV_FORMAT = env("HELIOS_HTTPS_SRV_FORMAT", "_%s._https.%s");
   private static final String HTTP_SRV_FORMAT = env("HELIOS_HTTP_SRV_FORMAT", "_%s._http.%s");
 
@@ -83,7 +87,9 @@ private static List<URI> resolve(final String srvName,
       endpoints.add(protocol(protocol, result.host(), result.port()));
     }
 
-    return endpoints.build();
+    final ImmutableList<URI> uris = endpoints.build();
+    log.info("Resolved {} to {}", name, uris);
+    return uris;
   }
 
   private static URI protocol(final String protocol, final String host, final int port) {