[ 
https://issues.apache.org/jira/browse/HADOOP-17290?focusedWorklogId=614683&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-614683
 ]

ASF GitHub Bot logged work on HADOOP-17290:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 24/Jun/21 19:09
            Start Date: 24/Jun/21 19:09
    Worklog Time Spent: 10m 
      Work Description: sumangala-patki commented on a change in pull request 
#2520:
URL: https://github.com/apache/hadoop/pull/2520#discussion_r658212957



##########
File path: 
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsRestOperation.java
##########
@@ -221,17 +226,27 @@ private void completeExecute() throws 
AzureBlobFileSystemException {
     LOG.trace("{} REST operation complete", operationType);
   }
 
+  private void updateClientRequestHeader(AbfsHttpOperation httpOperation,
+      TracingContext tracingContext) {
+    tracingContext.generateClientRequestID();

Review comment:
       Generating the clientRequestId in the toString() method of 
tracingContext is not a good option as it will re-generate it every time the 
method is called. Also, we need tracingContext to store the id for consistency
   Have renamed toString to constructHeader. Format is passed over config and 
already available to tracingContext object




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 614683)
    Time Spent: 5h 40m  (was: 5.5h)

> ABFS: Add Identifiers to Client Request Header
> ----------------------------------------------
>
>                 Key: HADOOP-17290
>                 URL: https://issues.apache.org/jira/browse/HADOOP-17290
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/azure
>    Affects Versions: 3.3.0
>            Reporter: Sumangala Patki
>            Assignee: Sumangala Patki
>            Priority: Major
>              Labels: abfsactive, pull-request-available
>          Time Spent: 5h 40m
>  Remaining Estimate: 0h
>
> Adding unique values to the client request header to assist in correlating 
> requests



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to