(hbase) branch master 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 master
in repository https://gitbox.apache.org/repos/asf/hbase.git


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

commit b8d803c0f1156219cc965e4c749e7ab7c9a65f31
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 6a6baab97ad..8b12c384087 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 7b03bd59530..a3ffbd858d8 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);
     SingleProcessHBaseCluster cluster = TEST_UTIL.getHBaseCluster();
     rs0 = cluster.getRegionServer(0).getServerName();