This is an automated email from the ASF dual-hosted git repository.
apurtell pushed a commit to branch branch-2.4
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.4 by this push:
new 368d52f HBASE-25840 CatalogJanitor warns about skipping gc of regions
during RIT, but does not actually skip (#3223)
368d52f is described below
commit 368d52fe99b08752358748d92935fefa1c30c467
Author: Andrew Purtell <[email protected]>
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 <[email protected]>
---
.../java/org/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 c23ccf0..dd3b6fc 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<RegionInfo, Result> mergedRegions = this.lastReport.mergedRegions;
for (Map.Entry<RegionInfo, Result> e : mergedRegions.entrySet()) {
if (this.services.isInMaintenanceMode()) {
