Skip to content

Commit 3f40df8

Browse files
committed
HBASE-22325 AsyncRpcRetryingCaller will not schedule retry if we hit a NotServingRegionException but there is no TableName provided
1 parent f9f6354 commit 3f40df8

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncRpcRetryingCaller.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ protected final void resetCallTimeout() {
126126
resetController(controller, callTimeoutNs, priority);
127127
}
128128

129-
private void tryScheduleRetry(Throwable error, Consumer<Throwable> updateCachedLocation) {
129+
private void tryScheduleRetry(Throwable error) {
130130
long pauseNsToUse = error instanceof CallQueueTooBigException ? pauseForCQTBENs : pauseNs;
131131
long delayNs;
132132
if (operationTimeoutNs > 0) {
@@ -192,19 +192,21 @@ protected final void onError(Throwable t, Supplier<String> errMsg,
192192
future.completeExceptionally(e);
193193
} else {
194194
// failed to test whether the table is disabled, not a big deal, continue retrying
195-
tryScheduleRetry(error, updateCachedLocation);
195+
tryScheduleRetry(error);
196196
}
197197
return;
198198
}
199199
if (disabled) {
200200
future.completeExceptionally(new TableNotEnabledException(tableName.get()));
201201
} else {
202-
tryScheduleRetry(error, updateCachedLocation);
202+
tryScheduleRetry(error);
203203
}
204204
});
205+
} else {
206+
tryScheduleRetry(error);
205207
}
206208
} else {
207-
tryScheduleRetry(error, updateCachedLocation);
209+
tryScheduleRetry(error);
208210
}
209211
}
210212

0 commit comments

Comments
 (0)