anujmodi2021 commented on code in PR #5881:
URL: https://github.com/apache/hadoop/pull/5881#discussion_r1308420397
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsRestOperation.java:
##########
@@ -224,15 +226,18 @@ private void completeExecute(TracingContext
tracingContext)
requestHeaders.add(httpHeader);
}
+ // By Default Exponential Retry Policy Will be used
retryCount = 0;
+ retryPolicy = client.getExponentialRetryPolicy();
LOG.debug("First execution of REST operation - {}", operationType);
while (!executeHttpOperation(retryCount, tracingContext)) {
try {
++retryCount;
tracingContext.setRetryCount(retryCount);
- LOG.debug("Retrying REST operation {}. RetryCount = {}",
- operationType, retryCount);
- Thread.sleep(client.getRetryPolicy().getRetryInterval(retryCount));
+ long retryInterval = retryPolicy.getRetryInterval(retryCount);
+ LOG.debug("Rest operation {} failed with failureReason: {}. Retrying
with retryCount = {}, retryPolicy: {} and sleepInterval: {}",
Review Comment:
We already have URL logged inside executeHttpOperation() function. In case
of retriable failures, URL will be logged just before this log.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
