(hbase) branch branch-2.5 updated: HBASE-29714 Increase DEFAULT_RS_REMOTE_PROC_RETRY_LIMIT to 10 (#7466)

[vjasani]

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

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


The following commit(s) were added to refs/heads/branch-2.5 by this push:
     new 13233bb9bd9 HBASE-29714 Increase DEFAULT_RS_REMOTE_PROC_RETRY_LIMIT to 
10 (#7466)
13233bb9bd9 is described below

commit 13233bb9bd98d110eb229a932f48b0cb15c2f5e0
Author: Umesh <9414umeshku...@gmail.com>
AuthorDate: Mon Nov 17 11:51:30 2025 +0530

    HBASE-29714 Increase DEFAULT_RS_REMOTE_PROC_RETRY_LIMIT to 10 (#7466)
    
    Signed-off-by: Andrew Purtell <apurt...@apache.org>
    Signed-off-by: Viraj Jasani <vjas...@apache.org>
    Signed-off-by: Aman Poonia <aman.poonia...@gmail.com>
---
 .../apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java   | 4 ++--
 .../test/java/org/apache/hadoop/hbase/util/TestProcDispatcher.java    | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java
index 90c47bda7ef..f92bdf4ed18 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java
@@ -261,9 +261,9 @@ public class RSProcedureDispatcher extends 
RemoteProcedureDispatcher<MasterProce
     /**
      * The default retry limit. Waiting for more than {@value} attempts is not 
going to help much
      * for genuine connectivity errors. Therefore, consider fail-fast after 
{@value} retries. Value
-     * = {@value}
+     * = {@value}. 10 retries means we will wait for at least 28.5 seconds 
before killing RS.
      */
-    private static final int DEFAULT_RS_REMOTE_PROC_RETRY_LIMIT = 5;
+    private static final int DEFAULT_RS_REMOTE_PROC_RETRY_LIMIT = 10;
 
     private final int failFastRetryLimit;
 
diff --git 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestProcDispatcher.java
 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestProcDispatcher.java
index 740a65f2b61..5e42ee2f4a4 100644
--- 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestProcDispatcher.java
+++ 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestProcDispatcher.java
@@ -77,6 +77,7 @@ public class TestProcDispatcher {
   public static void setUpBeforeClass() throws Exception {
     TEST_UTIL.getConfiguration().set(HBASE_MASTER_RSPROC_DISPATCHER_CLASS,
       RSProcDispatcher.class.getName());
+    
TEST_UTIL.getConfiguration().setInt("hbase.master.rs.remote.proc.fail.fast.limit",
 5);
     TEST_UTIL.startMiniCluster(3);
     MiniHBaseCluster cluster = TEST_UTIL.getHBaseCluster();
     rs0 = cluster.getRegionServer(0).getServerName();