Skip to content

Commit c0760d3

Browse files
chaijunjie0101Apache9
authored andcommitted
HBASE-27718 The regionStateNode only need remove once in regionOffline (#5106)
Signed-off-by: Duo Zhang <[email protected]> (cherry picked from commit e6977a9)
1 parent e68d0cf commit c0760d3

File tree

2 files changed

+0
-7
lines changed

2 files changed

+0
-7
lines changed

hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1043,8 +1043,6 @@ public void deleteTable(final TableName tableName) throws IOException {
10431043
regionStateStore.deleteRegions(regions);
10441044
for (int i = 0; i < regions.size(); ++i) {
10451045
final RegionInfo regionInfo = regions.get(i);
1046-
// we expect the region to be offline
1047-
regionStates.removeFromOfflineRegions(regionInfo);
10481046
regionStates.deleteRegion(regionInfo);
10491047
}
10501048
}

hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionStates.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -680,11 +680,6 @@ public void addToOfflineRegions(final RegionStateNode regionNode) {
680680
regionOffline.put(regionNode.getRegionInfo(), regionNode);
681681
}
682682

683-
// TODO: Unused.
684-
public void removeFromOfflineRegions(final RegionInfo regionInfo) {
685-
regionOffline.remove(regionInfo);
686-
}
687-
688683
// ==========================================================================
689684
// Region FAIL_OPEN helpers
690685
// ==========================================================================

0 commit comments

Comments
 (0)