Review comments.

Author: kannanjgithub

rls/src/main/java/io/grpc/rls/CachingRlsLbClient.java

@@ -230,7 +230,6 @@ private CachingRlsLbClient(Builder builder) {
     // TODO(creamsoup) wait until lb is ready
     String defaultTarget = lbPolicyConfig.getRouteLookupConfig().defaultTarget();
     if (defaultTarget != null && !defaultTarget.isEmpty()) {
-      logger.log(ChannelLogLevel.DEBUG, "starting fallback to {0}", defaultTarget);
       fallbackChildPolicyWrapper = refCountedChildPolicyWrapperFactory.createOrGet(defaultTarget);
     } else {
       fallbackChildPolicyWrapper = null;

rls/src/test/java/io/grpc/rls/CachingRlsLbClientTest.java

@@ -169,7 +169,6 @@ public void uncaughtException(Thread t, Throwable e) {
   private CachingRlsLbClient rlsLbClient;
   private Map<String, ?> rlsChannelServiceConfig;
   private String rlsChannelOverriddenAuthority;
-  private boolean isAlreadyClosed = false;
 
   private void setUpRlsLbClient() {
     fakeThrottler.resetCounts();
@@ -192,7 +191,7 @@ public void setUpMockMetricRecorder() {
 
   @After
   public void tearDown() throws Exception {
-    if (!isAlreadyClosed) {
+    if (rlsLbClient != null) {
       rlsLbClient.close();
     }
     assertWithMessage(
@@ -704,8 +703,8 @@ public void metricGauges() throws ExecutionException, InterruptedException, Time
 
     // Shutdown
     rlsLbClient.close();
+    rlsLbClient = null;
     verify(mockGaugeRegistration).close();
-    isAlreadyClosed = true;
   }
 
   private static RouteLookupConfig getRouteLookupConfig() {