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

Posted to user@hbase.apache.org by Gaojinchao <ga...@huawei.com> on 2011/03/04 10:11:13 UTC

It is a bug in function balanceCluster

In balanceCluster function , It should be "leastloaded=" + serversByLoad.firstKey ().getLoad().getNumberOfRegions())"


if(serversByLoad.lastKey().getLoad().getNumberOfRegions() <= max &&
       serversByLoad.firstKey().getLoad().getNumberOfRegions() >= min) {
      // Skipped because no server outside (min,max) range
      LOG.info("Skipping load balancing.  servers=" + numServers + " " +
          "regions=" + numRegions + " average=" + average + " " +
          "mostloaded=" + serversByLoad.lastKey().getLoad().getNumberOfRegions() +
          " leastloaded=" + serversByLoad.lastKey().getLoad().getNumberOfRegions());
      return null;
    }

Re: It is a bug in function balanceCluster

Posted by Stack <st...@duboce.net>.

Committed to TRUNK (HBASE-3605).  Thanks for pointing out the error.
St.Ack

On Fri, Mar 4, 2011 at 1:11 AM, Gaojinchao <ga...@huawei.com> wrote:
> In balanceCluster function , It should be "leastloaded=" + serversByLoad.firstKey ().getLoad().getNumberOfRegions())"
>
>
> if(serversByLoad.lastKey().getLoad().getNumberOfRegions() <= max &&
>       serversByLoad.firstKey().getLoad().getNumberOfRegions() >= min) {
>      // Skipped because no server outside (min,max) range
>      LOG.info("Skipping load balancing.  servers=" + numServers + " " +
>          "regions=" + numRegions + " average=" + average + " " +
>          "mostloaded=" + serversByLoad.lastKey().getLoad().getNumberOfRegions() +
>          " leastloaded=" + serversByLoad.lastKey().getLoad().getNumberOfRegions());
>      return null;
>    }
>