wchevreuil commented on code in PR #5273:
URL: https://github.com/apache/hbase/pull/5273#discussion_r1235122958
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java:
##########
@@ -1638,8 +1638,8 @@ private static class CompactionChecker extends
ScheduledChore {
@Override
protected void chore() {
for (Region r : this.instance.onlineRegions.values()) {
- // Skip compaction if region is read only
- if (r == null || r.isReadOnly()) {
+ // Skip compaction if region is read only or table disable compaction
Review Comment:
Let's be more clear:
`// If region is read only or compaction is disabled at table level, there's
no need to iterate through region's stores`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
