virajjasani commented on code in PR #5391:
URL: https://github.com/apache/hbase/pull/5391#discussion_r1323678057
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java:
##########
@@ -306,6 +308,10 @@ private boolean scheduleForRetry(IOException e) {
serverName, e.toString(), numberOfAttemptsSoFar);
return false;
}
+ if (isSaslError(e) && numberOfAttemptsSoFar == 0) {
Review Comment:
Duo's point is that, if some attempts were already successful, it might have
triggered region transition already and we might be in the middle of another
sub-procedure when we encounter this.
I also think num of attempts should not matter as we won't be able to make
any progress anyways, but then it takes our discussion back to the parent Jira
HBASE-28048
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
