[hbase] branch branch-2.3 updated: HBASE-25840 CatalogJanitor warns about skipping gc of regions during RIT, but does not actually skip (#3223)

[apurtell]

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

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


The following commit(s) were added to refs/heads/branch-2.3 by this push:
     new 0fa9dc2  HBASE-25840 CatalogJanitor warns about skipping gc of regions 
during RIT, but does not actually skip (#3223)
0fa9dc2 is described below

commit 0fa9dc2519d1f76cf7b187360b054ef7015058eb
Author: Andrew Purtell <apurt...@apache.org>
AuthorDate: Mon May 3 18:14:38 2021 -0700

    HBASE-25840 CatalogJanitor warns about skipping gc of regions during RIT, 
but does not actually skip (#3223)
    
    We claim in a WARN level log line to be "Playing-it-safe skipping merge/
    split gc'ing of regions from hbase:meta while regions-in-transition (RIT)"
    but do not actually skip because of a missing return. Remove the warning.
    
    Signed-off-by: Duo Zhang <zhang...@apache.org>
---
 .../src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java  | 4 ----
 1 file changed, 4 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java
index 9adae94..9ad4d5f 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java
@@ -176,10 +176,6 @@ public class CatalogJanitor extends ScheduledChore {
         LOG.warn(this.lastReport.toString());
       }
 
-      if (isRIT(this.services.getAssignmentManager())) {
-        LOG.warn("Playing-it-safe skipping merge/split gc'ing of regions from 
hbase:meta while " +
-            "regions-in-transition (RIT)");
-      }
       Map<RegionInfo, Result> mergedRegions = this.lastReport.mergedRegions;
       for (Map.Entry<RegionInfo, Result> e : mergedRegions.entrySet()) {
         if (this.services.isInMaintenanceMode()) {