HBASE-27718 The regionStateNode only need remove once in regionOffline (

apache#5106) Signed-off-by: Duo Zhang <zhangduo@apache.org> (cherry picked from commit e6977a9)
sunhelly · Mar 19, 2023 · c0760d3 · c0760d3
1 parent e68d0cf
commit c0760d3
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 7 deletions.
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
@@ -1043,8 +1043,6 @@ public void deleteTable(final TableName tableName) throws IOException {
     regionStateStore.deleteRegions(regions);
     for (int i = 0; i < regions.size(); ++i) {
       final RegionInfo regionInfo = regions.get(i);
-      // we expect the region to be offline
-      regionStates.removeFromOfflineRegions(regionInfo);
       regionStates.deleteRegion(regionInfo);
     }
   }

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionStates.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionStates.java
@@ -680,11 +680,6 @@ public void addToOfflineRegions(final RegionStateNode regionNode) {
     regionOffline.put(regionNode.getRegionInfo(), regionNode);
   }
 
-  // TODO: Unused.
-  public void removeFromOfflineRegions(final RegionInfo regionInfo) {
-    regionOffline.remove(regionInfo);
-  }
-
   // ==========================================================================
   // Region FAIL_OPEN helpers
   // ==========================================================================