sumangala-patki commented on a change in pull request #2520:
URL: https://github.com/apache/hadoop/pull/2520#discussion_r660083909
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsLease.java
##########
@@ -114,13 +119,15 @@ public AbfsLease(AbfsClient client, String path, int
acquireMaxRetries,
LOG.debug("Acquired lease {} on {}", leaseID, path);
}
- private void acquireLease(RetryPolicy retryPolicy, int numRetries, int
retryInterval, long delay)
+ private void acquireLease(RetryPolicy retryPolicy, int numRetries,
+ int retryInterval, long delay)
throws LeaseException {
LOG.debug("Attempting to acquire lease on {}, retry {}", path, numRetries);
if (future != null && !future.isDone()) {
throw new LeaseException(ERR_LEASE_FUTURE_EXISTS);
}
- future = client.schedule(() -> client.acquireLease(path,
INFINITE_LEASE_DURATION),
+ future = client.schedule(() -> client.acquireLease(path,
+ INFINITE_LEASE_DURATION, new TracingContext(tracingContext)),
Review comment:
not needed; moved to calling method (constructor) where it is cloned
only once, instead of per retry
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
