incubator-accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From e..@apache.org
Subject svn commit: r1245029 - /incubator/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
Date Thu, 16 Feb 2012 15:34:15 GMT
Author: ecn
Date: Thu Feb 16 15:34:14 2012
New Revision: 1245029

URL: http://svn.apache.org/viewvc?rev=1245029&view=rev
Log:
ACCUMULO-393: use the spiffy "retainAll" method to do the set intersection

Modified:
    incubator/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java

Modified: incubator/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
URL: http://svn.apache.org/viewvc/incubator/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java?rev=1245029&r1=1245028&r2=1245029&view=diff
==============================================================================
--- incubator/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
(original)
+++ incubator/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
Thu Feb 16 15:34:14 2012
@@ -1420,12 +1420,7 @@ public class Master implements Listener,
       }
     }
     synchronized (badServers) {
-      Set<TServerInstance> deadBadServers = new HashSet<TServerInstance>(badServers.keySet());
-      deadBadServers.removeAll(currentServers);
-      if (!badServers.isEmpty()) {
-        log.debug("Forgetting about bad servers: " + badServers);
-      }
-      badServers.entrySet().removeAll(deadBadServers);
+      badServers.keySet().retainAll(currentServers);
     }
     log.debug(String.format("Finished gathering information from %d servers in %.2f seconds",
result.size(), (System.currentTimeMillis() - start) / 1000.));
     return result;



Mime
View raw message