anujmodi2021 commented on code in PR #6314:
URL: https://github.com/apache/hadoop/pull/6314#discussion_r1527924421
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystem.java:
##########
@@ -700,6 +698,18 @@ public synchronized void close() throws IOException {
if (isClosed) {
return;
}
+ if (abfsStore.getClient().isMetricCollectionEnabled()) {
+ TracingContext tracingMetricContext = new TracingContext(
+ clientCorrelationId,
+ fileSystemId, FSOperationType.GET_ATTR, true,
Review Comment:
Here for metric tracing context we are using fileSystemId only, where as the
metric tracing context created in ABfsClient, we are suing hostname...
Any reason for this difference?
--
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]
