[hbase] branch branch-2.2 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.2
in repository https://gitbox.apache.org/repos/asf/hbase.git


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

commit 2ec2a0a0f275546d5ee7621a2eabdece3429ca24
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 fcd89af..da07426 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
@@ -174,10 +174,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()) {