You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 19:42:07 UTC

svn commit: r1181921 - /hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/loadtest/RegionSplitter.java

Author: nspiegelberg
Date: Tue Oct 11 17:42:07 2011
New Revision: 1181921

URL: http://svn.apache.org/viewvc?rev=1181921&view=rev
Log:
Add precondition check to regionsplitter for sanity checking

Modified:
    hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/loadtest/RegionSplitter.java

Modified: hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/loadtest/RegionSplitter.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/loadtest/RegionSplitter.java?rev=1181921&r1=1181920&r2=1181921&view=diff
==============================================================================
--- hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/loadtest/RegionSplitter.java (original)
+++ hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/loadtest/RegionSplitter.java Tue Oct 11 17:42:07 2011
@@ -290,6 +290,7 @@ public class RegionSplitter {
         // sort the RS by the number of regions they have
         List<HServerAddress> serversLeft = Lists.newArrayList(daughterRegions
             .keySet());
+        Preconditions.checkState(rsSizes.keySet().containsAll(serversLeft));
         Collections.sort(serversLeft, new Comparator<HServerAddress>() {
           public int compare(HServerAddress o1, HServerAddress o2) {
             return rsSizes.get(o1).compareTo(rsSizes.get(o2));