HBASE-615 Region balancer oscillates during cluster startup

-Change HServerLoad's getLoad method to ignore the number of requests, thus causing RegionManager to assign based merely on number of regions per server

git-svn-id: https://svn.apache.org/repos/asf/hadoop/hbase/trunk@669533 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -61,6 +61,7 @@ Hbase Change Log
    HBASE-683   can not get svn revision # at build time if locale is not english
                (Rong-En Fan via Stack)
    HBASE-699   Fix TestMigrate up on Hudson
+   HBASE-615   Region balancer oscillates during cluster startup
    
   IMPROVEMENTS
    HBASE-559   MR example job to count table rows

src/java/org/apache/hadoop/hbase/HServerLoad.java

@@ -59,12 +59,19 @@ public class HServerLoad implements WritableComparable {
   }
   
   /**
+   * Originally, this method factored in the effect of requests going to the
+   * server as well. However, this does not interact very well with the current
+   * region rebalancing code, which only factors number of regions. For the 
+   * interim, until we can figure out how to make rebalancing use all the info
+   * available, we're just going to make load purely the number of regions.
+   *
    * @return load factor for this server
    */
   public int getLoad() {
-    int load = numberOfRequests == 0 ? 1 : numberOfRequests;
+    // int load = numberOfRequests == 0 ? 1 : numberOfRequests;
-    load *= numberOfRegions == 0 ? 1 : numberOfRegions;
+    // load *= numberOfRegions == 0 ? 1 : numberOfRegions;
-    return load;
+    // return load;
+    return numberOfRegions;
   }
   
   /** {@inheritDoc} */