anujmodi2021 commented on code in PR #7837:
URL: https://github.com/apache/hadoop/pull/7837#discussion_r2245426192
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/utils/TracingContext.java:
##########
@@ -265,6 +289,34 @@ private String addFailureReasons(final String header,
return String.format("%s_%s", header, previousFailure);
}
+ private String getRetryHeader(final String previousFailure, String
retryPolicyAbbreviation) {
+ String retryHeader = String.format("%d", retryCount);
+ if (previousFailure == null) {
+ return retryHeader;
+ }
+ if (CONNECTION_TIMEOUT_ABBREVIATION.equals(previousFailure) &&
retryPolicyAbbreviation != null) {
+ return String.format("%s_%s_%s", retryHeader, previousFailure,
retryPolicyAbbreviation);
+ }
+ return String.format("%s_%s", retryHeader, previousFailure);
+ }
+
+ private String getOperationSpecificHeader(FSOperationType opType) {
+ // Similar header can be added for other operations in the future.
+ switch (opType) {
+ case READ:
+ return readSpecificHeader();
+ default:
+ return EMPTY_STRING; // no operation specific header
+ }
+ }
+
+ private String readSpecificHeader() {
+ // More information on read can be added to this header in the future.
+ // As underscore separated values.
+ String readHeader = String.format("%s", readType.toString());
Review Comment:
Rataining it as in future we might add more info to the same field
--
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
