accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From e..@apache.org
Subject svn commit: r1325429 - /accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
Date Thu, 12 Apr 2012 18:34:23 GMT
Author: ecn
Date: Thu Apr 12 18:34:22 2012
New Revision: 1325429

URL: http://svn.apache.org/viewvc?rev=1325429&view=rev
Log:
ACCUMULO-529 remove should have been removeAll

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

Modified: accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
URL: http://svn.apache.org/viewvc/accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java?rev=1325429&r1=1325428&r2=1325429&view=diff
==============================================================================
--- accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
(original)
+++ accumulo/branches/1.3/src/server/src/main/java/org/apache/accumulo/server/master/Master.java
Thu Apr 12 18:34:22 2012
@@ -1250,7 +1250,7 @@ public class Master implements Listener,
       synchronized (migrations) {
         notFound.addAll(migrations.keySet());
       }
-      notFound.remove(found);
+      notFound.removeAll(found);
       for (KeyExtent extent : notFound) {
         log.info("Canceling migration of " + extent + " to " + migrations.get(extent) + ":
tablet no longer exists (probably due to a split)");
         migrations.remove(extent);



Mime
View raw message