(hbase) branch branch-3 updated: HBASE-28332 Type conversion is no need in method CompactionChecker.chore() (#5653)

[zhangduo]

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

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


The following commit(s) were added to refs/heads/branch-3 by this push:
     new 324fe16abf3 HBASE-28332 Type conversion is no need in method 
CompactionChecker.chore() (#5653)
324fe16abf3 is described below

commit 324fe16abf3b0fcc7a02b3f88911e587ddbdd771
Author: guluo <lupeng_n...@qq.com>
AuthorDate: Fri Jan 26 18:17:22 2024 +0800

    HBASE-28332 Type conversion is no need in method CompactionChecker.chore() 
(#5653)
    
    Signed-off-by: Duo Zhang <zhang...@apache.org>
    (cherry picked from commit addb4e7fc3c0b6e486281033499b6de1f368d686)
---
 .../java/org/apache/hadoop/hbase/regionserver/HRegionServer.java   | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
index a77fa0cd879..dfb8e2a204f 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
@@ -1684,14 +1684,13 @@ public class HRegionServer extends 
HBaseServerBase<RSRpcServices>
 
     @Override
     protected void chore() {
-      for (Region r : this.instance.onlineRegions.values()) {
+      for (HRegion hr : this.instance.onlineRegions.values()) {
         // If region is read only or compaction is disabled at table level, 
there's no need to
         // iterate through region's stores
-        if (r == null || r.isReadOnly() || 
!r.getTableDescriptor().isCompactionEnabled()) {
+        if (hr == null || hr.isReadOnly() || 
!hr.getTableDescriptor().isCompactionEnabled()) {
           continue;
         }
 
-        HRegion hr = (HRegion) r;
         for (HStore s : hr.stores.values()) {
           try {
             long multiplier = s.getCompactionCheckMultiplier();
@@ -1719,7 +1718,7 @@ public class HRegionServer extends 
HBaseServerBase<RSRpcServices>
               }
             }
           } catch (IOException e) {
-            LOG.warn("Failed major compaction check on " + r, e);
+            LOG.warn("Failed major compaction check on " + hr, e);
           }
         }
       }