Skip to content

Commit 7973022

Browse files
author
Liudmila Molkova
authored
Change Diagnostic-Id spelling to comply with other languages (Azure#23141)
1 parent cc20cb4 commit 7973022

File tree

4 files changed

+12
-2
lines changed

4 files changed

+12
-2
lines changed

eng/code-quality-reports/src/main/resources/revapi/revapi.json

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -628,6 +628,12 @@
628628
"code": "java.method.added",
629629
"new": "method boolean com.google.gson.JsonArray::isEmpty()",
630630
"justification": "com.google.code.gson:gson:jar:2.8.7 added this new method."
631+
},
632+
{
633+
"code": "java.field.constantValueChanged",
634+
"old": "field com.azure.core.util.tracing.Tracer.DIAGNOSTIC_ID_KEY",
635+
"new": "field com.azure.core.util.tracing.Tracer.DIAGNOSTIC_ID_KEY",
636+
"justification": "Old field value spelled incorrectly and was used in preview version of tracing package"
631637
}
632638
]
633639
}

sdk/core/azure-core/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
- Added a new constructor `HttpHeaders(int initialCapacity)` which allows configuration of the initial backing map
77
capacity may allow short-circuiting scenarios where the map would need to be resized and copied in memory.
88

9+
### Fixed
10+
11+
- Fixed a bug with context propagation through EventHub and ServiceBus between Java and other languages.
12+
913
## 1.19.0-beta.1 (2021-07-07)
1014

1115
### Features Added

sdk/core/azure-core/src/main/java/com/azure/core/util/tracing/Tracer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public interface Tracer {
4848
/**
4949
* Key for {@link Context} which indicates that the context contains a "Diagnostic Id" for the service call.
5050
*/
51-
String DIAGNOSTIC_ID_KEY = "diagnostic-id";
51+
String DIAGNOSTIC_ID_KEY = "Diagnostic-Id";
5252

5353
/**
5454
* Key for {@link Context} the scope of code where the given Span is in the current Context.

sdk/core/azure-core/src/samples/java/com/azure/core/util/tracing/TracerJavaDocCodeSnippets.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public void startTracingSpan() {
4141
// END: com.azure.core.util.tracing.start#string-context-processKind-SEND
4242

4343
// BEGIN: com.azure.core.util.tracing.start#string-context-processKind-MESSAGE
44-
String diagnosticIdKey = "diagnostic-id";
44+
String diagnosticIdKey = "Diagnostic-Id";
4545
// start a new tracing span with explicit parent, sets the diagnostic Id (traceparent headers) on the current
4646
// context when process kind MESSAGE
4747
Context updatedReceiveContext = tracer.start("azure.eventhubs.receive", traceContext,

0 commit comments

Comments
 (0)