sguggilam commented on a change in pull request #2578:
URL: https://github.com/apache/hbase/pull/2578#discussion_r511241940



##########
File path: 
hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/RpcConnection.java
##########
@@ -126,8 +136,13 @@ protected RpcConnection(Configuration conf, 
HashedWheelTimer timeoutTimer, Conne
       LOG.debug("Use " + authMethod + " authentication for service " + 
remoteId.serviceName
           + ", sasl=" + useSasl);
     }
+
     reloginMaxBackoff = conf.getInt("hbase.security.relogin.maxbackoff", 5000);
     this.remoteId = remoteId;
+
+    forceReloginEnabled = 
conf.getBoolean("hbase.security.force.relogin.enabled", false);

Review comment:
       Yeah, that's a good point. we can actually default it to true and since 
the default minimum time between is anyways 10 mins which matches with that of 
the time in UGI for non forceful logins




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to