Skip to content

Commit 741789f

Browse files
mccullsdevflow.devflow-routing-intake
andauthored
Rename AgentSpan.context() to AgentSpan.spanContext() to disambiguate it from generic context (#11707)
Rename AgentSpan.context() to AgentSpan.spanContext() to disambiguate it from generic context Rename some AgentSpanContext variables for consistency Rename terminatedContextLinks -> terminatedSpanLinks Merge branch 'master' into mcculls/disambiguate-agent-span-context Co-authored-by: devflow.devflow-routing-intake <devflow.devflow-routing-intake@kubernetes.us1.ddbuild.io>
1 parent 2d6e0b0 commit 741789f

147 files changed

Lines changed: 442 additions & 425 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

dd-java-agent/agent-aiguard/src/main/java/com/datadog/aiguard/AIGuardInternal.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ public Evaluation evaluate(final List<Message> messages, final Options options)
269269
final AgentTracer.SpanBuilder builder = tracer.buildSpan(SPAN_NAME, SPAN_NAME);
270270
final AgentSpan parent = AgentTracer.activeSpan();
271271
if (parent != null) {
272-
builder.asChildOf(parent.context());
272+
builder.asChildOf(parent.spanContext());
273273
}
274274
final AgentSpan span = builder.start();
275275
final AgentSpan localRootSpan = span.getLocalRootSpan();

dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/instrumentation/decorator/BaseDecorator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ public AgentSpan afterStart(final AgentSpan span) {
100100
// DQH - Could retrieve the value from componentEntry and cast to avoid the virtual call,
101101
// unclear which option is better here
102102
final CharSequence component = component();
103-
span.context().setIntegrationName(component);
103+
span.spanContext().setIntegrationName(component);
104104

105105
// null handled by setMetric
106106
span.setMetric(traceAnalyticsEntry);

dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/instrumentation/decorator/HttpServerDecorator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ public AgentSpan onRequest(
434434
protected static AgentSpanContext.Extracted getExtractedSpanContext(Context parentContext) {
435435
AgentSpan extractedSpan = AgentSpan.fromContext(parentContext);
436436
if (extractedSpan != null) {
437-
AgentSpanContext extractedSpanContext = extractedSpan.context();
437+
AgentSpanContext extractedSpanContext = extractedSpan.spanContext();
438438
if (extractedSpanContext instanceof AgentSpanContext.Extracted) {
439439
return (AgentSpanContext.Extracted) extractedSpanContext;
440440
} else {

dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/instrumentation/decorator/WebsocketDecorator.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ private AgentSpan onFrameStart(
144144
wsSpan.setTag(DECISION_MAKER_RESOURCE, handshakeSpan.getResourceName());
145145
}
146146
} else {
147-
wsSpan = startSpan(WEBSOCKET.toString(), operationName, handshakeSpan.context());
147+
wsSpan = startSpan(WEBSOCKET.toString(), operationName, handshakeSpan.spanContext());
148148
}
149149
} else {
150150
wsSpan = startSpan(WEBSOCKET.toString(), operationName);
@@ -167,8 +167,8 @@ private AgentSpan onFrameStart(
167167
wsSpan.addLink(
168168
SpanLink.from(
169169
inheritSampling
170-
? handshakeSpan.context()
171-
: new NotSampledSpanContext(handshakeSpan.context()),
170+
? handshakeSpan.spanContext()
171+
: new NotSampledSpanContext(handshakeSpan.spanContext()),
172172
SpanLink.DEFAULT_FLAGS,
173173
"",
174174
linkAttributes));

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/BaseDecoratorTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ class BaseDecoratorTest extends DDSpecification {
3131
then:
3232
1 * span.setSpanType(decorator.spanType())
3333
1 * span.setTag(TagMap.Entry.create(Tags.COMPONENT, "test-component"))
34-
1 * span.context() >> spanContext
34+
1 * span.spanContext() >> spanContext
3535
1 * spanContext.setIntegrationName("test-component")
3636
_ * span.setTag(_)
3737
_ * span.setTag(_, _) // Want to allow other calls from child implementations.

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/ClientDecoratorTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class ClientDecoratorTest extends BaseDecoratorTest {
2424
}
2525
1 * span.setMeasured(true)
2626
1 * span.setTag(TagMap.Entry.create(Tags.COMPONENT, "test-component"))
27-
1 * span.context() >> spanContext
27+
1 * span.spanContext() >> spanContext
2828
1 * spanContext.setIntegrationName("test-component")
2929
1 * span.setTag(TagMap.Entry.create(Tags.SPAN_KIND, "client"))
3030
1 * span.setSpanType(decorator.spanType())

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/DBTypeProcessingDatabaseClientDecoratorTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ class DBTypeProcessingDatabaseClientDecoratorTest extends ClientDecoratorTest {
2525
}
2626
1 * span.setMeasured(true)
2727
1 * span.setTag(Tags.COMPONENT, "test-component")
28-
1 * span.context() >> spanContext
28+
1 * span.spanContext() >> spanContext
2929
1 * spanContext.setIntegrationName("test-component")
3030
1 * span.setTag(TagMap.Entry.create(Tags.SPAN_KIND, "client"))
3131
1 * span.setSpanType("test-type")

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/DatabaseClientDecoratorTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class DatabaseClientDecoratorTest extends ClientDecoratorTest {
2828
}
2929
1 * span.setMeasured(true)
3030
1 * span.setTag(TagMap.Entry.create(Tags.COMPONENT, "test-component"))
31-
1 * span.context() >> spanContext
31+
1 * span.spanContext() >> spanContext
3232
1 * spanContext.setIntegrationName("test-component")
3333
1 * span.setTag(TagMap.Entry.create(Tags.SPAN_KIND, "client"))
3434
1 * span.setSpanType("test-type")

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/ServerDecoratorTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class ServerDecoratorTest extends BaseDecoratorTest {
2424
then:
2525
1 * span.setTag(TagMap.Entry.create(LANGUAGE_TAG_KEY, LANGUAGE_TAG_VALUE))
2626
1 * span.setTag(TagMap.Entry.create(COMPONENT, "test-component"))
27-
1 * span.context() >> spanContext
27+
1 * span.spanContext() >> spanContext
2828
1 * spanContext.setIntegrationName("test-component")
2929
1 * span.setTag(TagMap.Entry.create(SPAN_KIND, "server"))
3030
1 * span.setSpanType(decorator.spanType())

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/decorator/TestDecoratorImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public AgentSpan afterStart(final AgentSpan span) {
5151
span.setTag(DDTags.HOST_VCPU_COUNT, cpuCount);
5252
span.setTag(Tags.TEST_TYPE, testType());
5353
span.setTag(Tags.COMPONENT, component());
54-
span.context().setIntegrationName(component());
54+
span.spanContext().setIntegrationName(component());
5555
span.setTag(Tags.TEST_SESSION_NAME, sessionName);
5656

5757
for (final Map.Entry<String, String> ciTag : ciTags.entrySet()) {

0 commit comments

Comments
 (0)