From 60055f555433bd6a69618e013eda9646da9b7aff Mon Sep 17 00:00:00 2001 From: Andrew Purtell Date: Mon, 3 May 2021 18:14:38 -0700 Subject: [PATCH] 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 --- .../apache/hadoop/hbase/master/janitor/CatalogJanitor.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/janitor/CatalogJanitor.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/janitor/CatalogJanitor.java index c23ccf0f58e..dd3b6fc32ff 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/janitor/CatalogJanitor.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/janitor/CatalogJanitor.java @@ -167,10 +167,6 @@ public class CatalogJanitor extends ScheduledChore { } updateAssignmentManagerMetrics(); - 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 mergedRegions = this.lastReport.mergedRegions; for (Map.Entry e : mergedRegions.entrySet()) { if (this.services.isInMaintenanceMode()) {