Skip to content

Commit 911f98c

Browse files
cbaenzigerpetersomogyi
authored andcommitted
HBASE-22487 getMostLoadedRegions is unused
Signed-off-by: Wellington Chevreuil <wellington.chevreuil@gmail.com> Signed-off-by: Peter Somogyi <psomogyi@apache.org>
1 parent f23c1c1 commit 911f98c

1 file changed

Lines changed: 0 additions & 21 deletions

File tree

hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -2585,27 +2585,6 @@ public FlushRequester getFlushRequester() {
25852585
return this.cacheFlusher;
25862586
}
25872587

2588-
/**
2589-
* Get the top N most loaded regions this server is serving so we can tell the
2590-
* master which regions it can reallocate if we're overloaded. TODO: actually
2591-
* calculate which regions are most loaded. (Right now, we're just grabbing
2592-
* the first N regions being served regardless of load.)
2593-
*/
2594-
protected HRegionInfo[] getMostLoadedRegions() {
2595-
ArrayList<HRegionInfo> regions = new ArrayList<HRegionInfo>();
2596-
for (Region r : onlineRegions.values()) {
2597-
if (!r.isAvailable()) {
2598-
continue;
2599-
}
2600-
if (regions.size() < numRegionsToReport) {
2601-
regions.add(r.getRegionInfo());
2602-
} else {
2603-
break;
2604-
}
2605-
}
2606-
return regions.toArray(new HRegionInfo[regions.size()]);
2607-
}
2608-
26092588
@Override
26102589
public Leases getLeases() {
26112590
return leases;

0 commit comments

Comments
 (0)