feat(core): Extract span and scope implementations from tracer class

Remove unused eligibleForDropping method
DataDog · Feb 1, 2025 · 2a4a5ad · 2a4a5ad
1 parent f677ee0
commit 2a4a5ad
Show file tree

Hide file tree

Showing 44 changed files with 943 additions and 774 deletions.
diff --git a/...tstrap/src/main/java/datadog/trace/bootstrap/instrumentation/api/Java8BytecodeBridge.java b/...tstrap/src/main/java/datadog/trace/bootstrap/instrumentation/api/Java8BytecodeBridge.java
@@ -0,0 +1,30 @@
+package datadog.trace.bootstrap.instrumentation.api;
+
+/**
+ * A helper for accessing methods that rely on new Java 8 bytecode features such as calling a static
+ * interface methods. In instrumentation, we may need to call these methods in code that is inlined
+ * into an instrumented class, however many times the instrumented class has been compiled to a
+ * previous version of bytecode, and so we cannot inline calls to static interface methods, as those
+ * were not supported prior to Java 8 and will lead to a class verification error.
+ */
+public class Java8BytecodeBridge {
+  /** @see AgentScope#noop() */
+  public static AgentScope noopScope() {
+    return AgentScope.noop();
+  }
+
+  /** @see AgentSpanContext#noop() */
+  public static AgentSpanContext noopSpanContext() {
+    return AgentSpanContext.noop();
+  }
+
+  /** @see AgentTracer#noopSpan() */
+  public static AgentSpan noopSpan() {
+    return AgentTracer.noopSpan();
+  }
+
+  /** @see AgentTracer#blackholeSpan() */
+  public static AgentSpan blackHoleSpan() {
+    return AgentTracer.blackholeSpan();
+  }
+}
diff --git a/...rc/main/java/datadog/trace/bootstrap/instrumentation/java/concurrent/ConcurrentState.java b/...rc/main/java/datadog/trace/bootstrap/instrumentation/java/concurrent/ConcurrentState.java
@@ -1,10 +1,10 @@
 package datadog.trace.bootstrap.instrumentation.java.concurrent;
 
+import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.noopSpan;
 import static datadog.trace.bootstrap.instrumentation.java.concurrent.ContinuationClaim.CLAIMED;
 
 import datadog.trace.bootstrap.ContextStore;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import java.util.concurrent.atomic.AtomicReferenceFieldUpdater;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -32,7 +32,7 @@ private ConcurrentState() {}
   public static <K> ConcurrentState captureScope(
       ContextStore<K, ConcurrentState> contextStore, K key, AgentScope scope) {
     if (scope != null && scope.isAsyncPropagating()) {
-      if (scope.span() instanceof AgentTracer.NoopAgentSpan) {
+      if (scope.span() == noopSpan()) {
         return null;
       }
       final ConcurrentState state = contextStore.putIfAbsent(key, FACTORY);

diff --git a/...main/java/datadog/trace/instrumentation/akka/concurrent/AkkaActorCellInstrumentation.java b/...main/java/datadog/trace/instrumentation/akka/concurrent/AkkaActorCellInstrumentation.java
@@ -4,6 +4,7 @@
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeScope;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeSpan;
+import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.noopSpan;
 import static java.util.Collections.singletonMap;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;
 
@@ -13,7 +14,6 @@
 import datadog.trace.agent.tooling.InstrumenterModule;
 import datadog.trace.bootstrap.InstrumentationContext;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import datadog.trace.bootstrap.instrumentation.java.concurrent.AdviceUtils;
 import datadog.trace.bootstrap.instrumentation.java.concurrent.State;
 import java.util.Map;
@@ -72,11 +72,11 @@ public static AgentScope enter(
         return null;
       }
       // If there is a noop span in the active scope, we can clean all the way to this scope
-      if (activeSpan() instanceof AgentTracer.NoopAgentSpan) {
+      if (activeSpan() == noopSpan()) {
         return activeScope;
       }
       // Create an active scope with a noop span, and clean all the way to the previous scope
-      localScope = activateSpan(AgentTracer.NoopAgentSpan.INSTANCE, false);
+      localScope = activateSpan(noopSpan(), false);
       return activeScope;
     }
 

diff --git a/...c/main/java/datadog/trace/instrumentation/akka/concurrent/AkkaMailboxInstrumentation.java b/...c/main/java/datadog/trace/instrumentation/akka/concurrent/AkkaMailboxInstrumentation.java
@@ -4,6 +4,7 @@
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeScope;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeSpan;
+import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.noopSpan;
 import static java.util.Collections.singletonList;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;
 
@@ -12,7 +13,6 @@
 import datadog.trace.agent.tooling.Instrumenter;
 import datadog.trace.agent.tooling.InstrumenterModule;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import datadog.trace.bootstrap.instrumentation.java.concurrent.ExcludeFilter;
 import java.util.Collection;
 import java.util.EnumMap;
@@ -69,11 +69,11 @@ public static AgentScope enter() {
         return null;
       }
       // If there is a noop span in the active scope, we can clean all the way to this scope
-      if (activeSpan() instanceof AgentTracer.NoopAgentSpan) {
+      if (activeSpan() == noopSpan()) {
         return activeScope;
       }
       // Create an active scope with a noop span, and clean all the way to the previous scope
-      activateSpan(AgentTracer.NoopAgentSpan.INSTANCE, false);
+      activateSpan(noopSpan(), false);
       return activeScope;
     }
 

diff --git a/...11.0/src/main/java/datadog/trace/instrumentation/aws/v0/AWSHttpClientInstrumentation.java b/...11.0/src/main/java/datadog/trace/instrumentation/aws/v0/AWSHttpClientInstrumentation.java
@@ -2,6 +2,7 @@
 
 import static datadog.trace.agent.tooling.bytebuddy.matcher.NameMatchers.named;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeScope;
+import static datadog.trace.bootstrap.instrumentation.api.Java8BytecodeBridge.noopSpan;
 import static datadog.trace.instrumentation.aws.v0.OnErrorDecorator.DECORATE;
 import static datadog.trace.instrumentation.aws.v0.OnErrorDecorator.SPAN_CONTEXT_KEY;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;
@@ -12,7 +13,6 @@
 import datadog.trace.agent.tooling.Instrumenter;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
 import datadog.trace.bootstrap.instrumentation.api.AgentSpan;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import net.bytebuddy.asm.Advice;
 
 /**
@@ -50,7 +50,7 @@ public static void methodExit(
       // check name in case TracingRequestHandler failed to activate the span
       if (scope != null
           && (AwsNameCache.spanName(request).equals(scope.span().getSpanName())
-              || scope.span() instanceof AgentTracer.NoopAgentSpan)) {
+              || scope.span() == noopSpan())) {
         scope.close();
       }
 

diff --git a/....0/src/main/java/datadog/trace/instrumentation/aws/v0/RequestExecutorInstrumentation.java b/....0/src/main/java/datadog/trace/instrumentation/aws/v0/RequestExecutorInstrumentation.java
@@ -2,6 +2,7 @@
 
 import static datadog.trace.agent.tooling.bytebuddy.matcher.NameMatchers.named;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeScope;
+import static datadog.trace.bootstrap.instrumentation.api.Java8BytecodeBridge.noopSpan;
 import static datadog.trace.instrumentation.aws.v0.OnErrorDecorator.DECORATE;
 import static datadog.trace.instrumentation.aws.v0.OnErrorDecorator.SPAN_CONTEXT_KEY;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;
@@ -10,7 +11,6 @@
 import datadog.trace.agent.tooling.Instrumenter;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
 import datadog.trace.bootstrap.instrumentation.api.AgentSpan;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import net.bytebuddy.asm.Advice;
 
 /**
@@ -47,7 +47,7 @@ public static void methodExit(
       // check name in case TracingRequestHandler failed to activate the span
       if (scope != null
           && (AwsNameCache.spanName(request).equals(scope.span().getSpanName())
-              || scope.span() instanceof AgentTracer.NoopAgentSpan)) {
+              || scope.span() == noopSpan())) {
         scope.close();
       }
 

diff --git a/...-2.2/src/main/java/datadog/trace/instrumentation/aws/v2/AwsHttpClientInstrumentation.java b/...-2.2/src/main/java/datadog/trace/instrumentation/aws/v2/AwsHttpClientInstrumentation.java
@@ -6,7 +6,7 @@
 import static datadog.trace.agent.tooling.bytebuddy.matcher.NameMatchers.namedOneOf;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeScope;
-import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.noopSpan;
+import static datadog.trace.bootstrap.instrumentation.api.Java8BytecodeBridge.noopSpan;
 import static net.bytebuddy.matcher.ElementMatchers.isMethod;
 import static net.bytebuddy.matcher.ElementMatchers.isPublic;
 import static net.bytebuddy.matcher.ElementMatchers.takesArgument;
@@ -15,7 +15,6 @@
 import datadog.trace.agent.tooling.Instrumenter;
 import datadog.trace.agent.tooling.InstrumenterModule;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import net.bytebuddy.asm.Advice;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
@@ -76,7 +75,7 @@ public static AgentScope methodEnter(
       final AgentScope scope = activeScope();
       // check name in case TracingExecutionInterceptor failed to activate the span
       if (scope != null
-          && (scope.span() instanceof AgentTracer.NoopAgentSpan
+          && (scope.span() == noopSpan()
               || AwsSdkClientDecorator.DECORATE
                   .spanName(requestExecutionContext.executionAttributes())
                   .equals(scope.span().getSpanName()))) {

diff --git a/...java/datadog/trace/instrumentation/netty40/NettyChannelHandlerContextInstrumentation.java b/...java/datadog/trace/instrumentation/netty40/NettyChannelHandlerContextInstrumentation.java
@@ -5,6 +5,7 @@
 import static datadog.trace.agent.tooling.bytebuddy.matcher.NameMatchers.named;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeSpan;
+import static datadog.trace.bootstrap.instrumentation.api.Java8BytecodeBridge.noopScope;
 import static datadog.trace.instrumentation.netty40.AttributeKeys.SPAN_ATTRIBUTE_KEY;
 import static datadog.trace.instrumentation.netty40.NettyChannelPipelineInstrumentation.ADDITIONAL_INSTRUMENTATION_NAMES;
 import static datadog.trace.instrumentation.netty40.NettyChannelPipelineInstrumentation.INSTRUMENTATION_NAME;
@@ -16,7 +17,6 @@
 import datadog.trace.agent.tooling.InstrumenterModule;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
 import datadog.trace.bootstrap.instrumentation.api.AgentSpan;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import datadog.trace.instrumentation.netty40.client.NettyHttpClientDecorator;
 import datadog.trace.instrumentation.netty40.server.NettyHttpServerDecorator;
 import io.netty.channel.ChannelHandlerContext;
@@ -72,7 +72,7 @@ public static AgentScope scopeSpan(@Advice.This final ChannelHandlerContext ctx)
       final AgentSpan channelSpan = ctx.channel().attr(SPAN_ATTRIBUTE_KEY).get();
       if (channelSpan == null || channelSpan == activeSpan()) {
         // don't modify the scope
-        return AgentTracer.NoopAgentScope.INSTANCE;
+        return noopScope();
       }
       return activateSpan(channelSpan);
     }

diff --git a/...java/datadog/trace/instrumentation/netty41/NettyChannelHandlerContextInstrumentation.java b/...java/datadog/trace/instrumentation/netty41/NettyChannelHandlerContextInstrumentation.java
@@ -5,6 +5,7 @@
 import static datadog.trace.agent.tooling.bytebuddy.matcher.NameMatchers.named;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activateSpan;
 import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.activeSpan;
+import static datadog.trace.bootstrap.instrumentation.api.Java8BytecodeBridge.noopScope;
 import static datadog.trace.instrumentation.netty41.AttributeKeys.SPAN_ATTRIBUTE_KEY;
 import static datadog.trace.instrumentation.netty41.NettyChannelPipelineInstrumentation.ADDITIONAL_INSTRUMENTATION_NAMES;
 import static datadog.trace.instrumentation.netty41.NettyChannelPipelineInstrumentation.INSTRUMENTATION_NAME;
@@ -16,7 +17,6 @@
 import datadog.trace.agent.tooling.InstrumenterModule;
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
 import datadog.trace.bootstrap.instrumentation.api.AgentSpan;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import datadog.trace.instrumentation.netty41.client.NettyHttpClientDecorator;
 import datadog.trace.instrumentation.netty41.server.NettyHttpServerDecorator;
 import io.netty.channel.ChannelHandlerContext;
@@ -72,7 +72,7 @@ public static AgentScope scopeSpan(@Advice.This final ChannelHandlerContext ctx)
       final AgentSpan channelSpan = ctx.channel().attr(SPAN_ATTRIBUTE_KEY).get();
       if (channelSpan == null || channelSpan == activeSpan()) {
         // don't modify the scope
-        return AgentTracer.NoopAgentScope.INSTANCE;
+        return noopScope();
       }
       return activateSpan(channelSpan);
     }

diff --git a/...elemetry-0.3/src/main/java/datadog/trace/instrumentation/opentelemetry/TypeConverter.java b/...elemetry-0.3/src/main/java/datadog/trace/instrumentation/opentelemetry/TypeConverter.java
@@ -1,9 +1,10 @@
 package datadog.trace.instrumentation.opentelemetry;
 
+import static datadog.trace.bootstrap.instrumentation.api.AgentTracer.noopSpan;
+
 import datadog.trace.bootstrap.instrumentation.api.AgentScope;
 import datadog.trace.bootstrap.instrumentation.api.AgentSpan;
 import datadog.trace.bootstrap.instrumentation.api.AgentSpanContext;
-import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import datadog.trace.bootstrap.instrumentation.api.AttachableWrapper;
 import io.opentelemetry.context.Scope;
 import io.opentelemetry.trace.Span;
@@ -16,16 +17,16 @@ public class TypeConverter {
   private final OtelScope noopScopeWrapper;
 
   public TypeConverter() {
-    noopSpanWrapper = new OtelSpan(AgentTracer.NoopAgentSpan.INSTANCE, this);
-    noopContextWrapper = new OtelSpanContext(AgentTracer.NoopContext.INSTANCE);
-    noopScopeWrapper = new OtelScope(AgentTracer.NoopAgentScope.INSTANCE);
+    noopSpanWrapper = new OtelSpan(noopSpan(), this);
+    noopContextWrapper = new OtelSpanContext(AgentSpanContext.noop());
+    noopScopeWrapper = new OtelScope(AgentScope.noop());
   }
 
   public AgentSpan toAgentSpan(final Span span) {
     if (span instanceof OtelSpan) {
       return ((OtelSpan) span).asAgentSpan();
     }
-    return null == span ? null : AgentTracer.NoopAgentSpan.INSTANCE;
+    return null == span ? null : noopSpan();
   }
 
   public Span toSpan(final AgentSpan agentSpan) {
@@ -42,7 +43,7 @@ public Span toSpan(final AgentSpan agentSpan) {
       attachableSpanWrapper.attachWrapper(spanWrapper);
       return spanWrapper;
     }
-    if (agentSpan == AgentTracer.NoopAgentSpan.INSTANCE) {
+    if (agentSpan == noopSpan()) {
       return noopSpanWrapper;
     }
     return new OtelSpan(agentSpan, this);
@@ -62,7 +63,7 @@ public Scope toScope(final AgentScope scope) {
       attachableScopeWrapper.attachWrapper(otScope);
       return otScope;
     }
-    if (scope == AgentTracer.NoopAgentScope.INSTANCE) {
+    if (scope == AgentScope.noop()) {
       return noopScopeWrapper;
     }
     return new OtelScope(scope);
@@ -73,7 +74,7 @@ public SpanContext toSpanContext(final AgentSpanContext context) {
       return null;
     }
     // avoid a new SpanContext wrapper allocation for the noop context
-    if (context == AgentTracer.NoopContext.INSTANCE) {
+    if (context == AgentSpanContext.noop()) {
       return noopContextWrapper;
     }
     return new OtelSpanContext(context);
@@ -83,6 +84,6 @@ public AgentSpanContext toContext(final SpanContext spanContext) {
     if (spanContext instanceof OtelSpanContext) {
       return ((OtelSpanContext) spanContext).getDelegate();
     }
-    return null == spanContext ? null : AgentTracer.NoopContext.INSTANCE;
+    return null == spanContext ? null : AgentSpanContext.noop();
   }
 }
diff --git a/.../instrumentation/opentelemetry/opentelemetry-0.3/src/test/groovy/TypeConverterTest.groovy b/.../instrumentation/opentelemetry/opentelemetry-0.3/src/test/groovy/TypeConverterTest.groovy
@@ -2,6 +2,8 @@ import datadog.trace.agent.test.AgentTestRunner
 import datadog.trace.api.DDSpanId
 import datadog.trace.api.DDTraceId
 import datadog.trace.api.sampling.PrioritySampling
+import datadog.trace.bootstrap.instrumentation.api.AgentScope
+import datadog.trace.bootstrap.instrumentation.api.AgentSpanContext
 import datadog.trace.bootstrap.instrumentation.api.AgentTracer
 import datadog.trace.bootstrap.instrumentation.api.AgentTracer.NoopPathwayContext
 import datadog.trace.bootstrap.instrumentation.api.ScopeSource
@@ -16,7 +18,7 @@ class TypeConverterTest extends AgentTestRunner {
   TypeConverter typeConverter = new TypeConverter()
 
   def "should avoid the noop span wrapper allocation"() {
-    def noopAgentSpan = AgentTracer.NoopAgentSpan.INSTANCE
+    def noopAgentSpan = AgentTracer.noopSpan()
     expect:
     typeConverter.toSpan(noopAgentSpan) is typeConverter.toSpan(noopAgentSpan)
   }
@@ -33,13 +35,13 @@ class TypeConverterTest extends AgentTestRunner {
   }
 
   def "should avoid the noop context wrapper allocation"() {
-    def noopContext = AgentTracer.NoopContext.INSTANCE
+    def noopContext = AgentSpanContext.noop()
     expect:
     typeConverter.toSpanContext(noopContext) is typeConverter.toSpanContext(noopContext)
   }
 
   def "should avoid the noop scope wrapper allocation"() {
-    def noopScope = AgentTracer.NoopAgentScope.INSTANCE
+    def noopScope = AgentScope.noop()
     expect:
     typeConverter.toScope(noopScope) is typeConverter.toScope(noopScope)
   }