Author: stack
Date: Thu Sep  9 05:43:13 2010
New Revision: 995318

URL: http://svn.apache.org/viewvc?rev=995318&view=rev
Log:
HBASE-2974 LoadBalancer ArithmeticException: / by zero

Modified:
    hbase/trunk/CHANGES.txt
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java

Modified: hbase/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=995318&r1=995317&r2=995318&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Thu Sep  9 05:43:13 2010
@@ -514,6 +514,7 @@ Release 0.21.0 - Unreleased
    HBASE-2969  missing sync in HTablePool.getTable()
                (Guilherme Mauro Germoglio Barbosa via Stack)
    HBASE-2973  NPE in LogCleaner
+   HBASE-2974  LoadBalancer ArithmeticException: / by zero
 
   IMPROVEMENTS
    HBASE-1760  Cleanup TODOs in HTable

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java?rev=995318&r1=995317&r2=995318&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java 
(original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java 
Thu Sep  9 05:43:13 2010
@@ -140,6 +140,10 @@ public class LoadBalancer {
       new TreeMap<HServerInfo,List<HRegionInfo>>(
           new HServerInfo.LoadComparator());
     int numServers = clusterState.size();
+    if (numServers == 0) {
+      LOG.debug("numServers=0 so nothing to balance");
+      return null;
+    }
     int numRegions = 0;
     // Iterate so we can count regions as we build the map
     for(Map.Entry<HServerInfo, List<HRegionInfo>> server :


Reply via email to