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

Posted to dev@hbase.apache.org by Gaojinchao <ga...@huawei.com> on 2011/05/12 13:00:55 UTC

a little bug :)

Hbase version: 0.90.2

" leastloaded=" + serversByLoad.lastKey().getLoad().getNumberOfRegions());

Should be:

" leastloaded=" + serversByLoad. firstKey ().getLoad().getNumberOfRegions());



if(serversByLoad.lastKey().getLoad().getNumberOfRegions() <= ceiling &&
       serversByLoad.firstKey().getLoad().getNumberOfRegions() >= floor) {
      // 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: a little bug :)

Posted by Jean-Daniel Cryans <jd...@apache.org>.

Fixed, thanks!

J-D

On Thu, May 12, 2011 at 4:00 AM, Gaojinchao <ga...@huawei.com> wrote:
> Hbase version: 0.90.2
>
> " leastloaded=" + serversByLoad.lastKey().getLoad().getNumberOfRegions());
>
> Should be:
>
> " leastloaded=" + serversByLoad. firstKey ().getLoad().getNumberOfRegions());
>
>
>
> if(serversByLoad.lastKey().getLoad().getNumberOfRegions() <= ceiling &&
>       serversByLoad.firstKey().getLoad().getNumberOfRegions() >= floor) {
>      // 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;
>    }
>