HBASE-29714 Increase DEFAULT_RS_REMOTE_PROC_RETRY_LIMIT to 10 (#7466)

Signed-off-by: Andrew Purtell <[email protected]>
Signed-off-by: Viraj Jasani <[email protected]>
Signed-off-by: Aman Poonia <[email protected]>

Author: Umeshkumar9414

hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java

@@ -261,9 +261,9 @@ protected class ExecuteProceduresRemoteCall implements RemoteProcedureResolver,
     /**
      * 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;
 

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();