a few cleanups

processing/src/main/java/io/druid/query/select/SelectQueryQueryToolChest.java

@@ -170,10 +170,9 @@ public class SelectQueryQueryToolChest extends QueryToolChest<Result<SelectResul
           ++index;
         }
 
-
         final Set<String> metrics = Sets.newTreeSet();
         if (query.getMetrics() != null) {
-          dimensions.addAll(query.getMetrics());
+          metrics.addAll(query.getMetrics());
         }
 
         final byte[][] metricBytes = new byte[metrics.size()][];

server/src/main/java/io/druid/curator/announcement/Announcer.java

@@ -212,11 +212,6 @@ public class Announcer
                         pathsToReinstate.add(path);
                       }
 
-                      for (ChildData data : cache.getCurrentData()) {
-                        log.info("Node[%s] in cache is removed to reinstate.", data.getPath());
-                        pathsToReinstate.remove(data.getPath());
-                      }
-
                       if (!pathsToReinstate.isEmpty() && !pathsLost.compareAndSet(null, pathsToReinstate)) {
                         log.info("Already had a pathsLost set!?[%s]", parentPath);
                       }