[hbase] branch branch-1 updated: HBASE-22487 getMostLoadedRegions is unused

[ps...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

psomogyi pushed a commit to branch branch-1
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-1 by this push:
     new 1b2f863  HBASE-22487 getMostLoadedRegions is unused
1b2f863 is described below

commit 1b2f863cdb807a5ff12e4f04cc0a7e27588a04ff
Author: Clay Baenziger <cw...@clayb.net>
AuthorDate: Mon May 27 21:57:15 2019 -0400

    HBASE-22487 getMostLoadedRegions is unused
    
    Signed-off-by: Wellington Chevreuil <we...@gmail.com>
    Signed-off-by: Peter Somogyi <ps...@apache.org>
---
 .../hadoop/hbase/regionserver/HRegionServer.java    | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
index fb1bbba..ab76e90 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
@@ -2657,27 +2657,6 @@ public class HRegionServer extends HasThread implements
     return this.cacheFlusher;
   }
 
-  /**
-   * Get the top N most loaded regions this server is serving so we can tell the
-   * master which regions it can reallocate if we're overloaded. TODO: actually
-   * calculate which regions are most loaded. (Right now, we're just grabbing
-   * the first N regions being served regardless of load.)
-   */
-  protected HRegionInfo[] getMostLoadedRegions() {
-    ArrayList<HRegionInfo> regions = new ArrayList<HRegionInfo>();
-    for (Region r : onlineRegions.values()) {
-      if (!r.isAvailable()) {
-        continue;
-      }
-      if (regions.size() < numRegionsToReport) {
-        regions.add(r.getRegionInfo());
-      } else {
-        break;
-      }
-    }
-    return regions.toArray(new HRegionInfo[regions.size()]);
-  }
-
   @Override
   public Leases getLeases() {
     return leases;