Skip to content

Commit 63abb7f

Browse files
Tracing header fix
1 parent a0d3ef9 commit 63abb7f

File tree

3 files changed

+1
-7
lines changed

3 files changed

+1
-7
lines changed

hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/utils/TracingMetricContext.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,9 +79,6 @@ public void constructHeader(AbfsHttpOperation httpOperation){
7979
header = "";
8080
break;
8181
}
82-
if (listener != null) { //for testing
83-
listener.callTracingHeaderValidator(header, tracingHeaderFormat);
84-
}
8582
httpOperation.setRequestProperty(HttpHeaderConfigurations.X_MS_CLIENT_REQUEST_ID, header);
8683
}
8784
}

hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/utils/TestMockHelpers.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,4 +56,4 @@ public static <T> T setClassField(
5656
private TestMockHelpers() {
5757
// Not called. - For checkstyle: HideUtilityClassConstructor
5858
}
59-
}
59+
}

hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/utils/TracingHeaderValidator.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,6 @@ private void validateBasicFormat(String[] idList) {
9797
} else if (format == TracingHeaderFormat.TWO_ID_FORMAT) {
9898
Assertions.assertThat(idList)
9999
.describedAs("header should have 2 elements").hasSize(2);
100-
} else if (format == TracingHeaderFormat.INTERNAL_METRIC_FORMAT) {
101-
Assertions.assertThat(idList)
102-
.describedAs("header should have 4 elements").hasSize(4);
103100
} else {
104101
Assertions.assertThat(idList).describedAs("header should have 1 element")
105102
.hasSize(1);

0 commit comments

Comments
 (0)