fix instrumentation dependency issues

solarwinds · Nov 12, 2021 · 0ee0b54 · 0ee0b54
1 parent 41ec815
commit 0ee0b54
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/...src/main/java/com/appoptics/opentelemetry/instrumentation/AoStatementInstrumentation.java b/...src/main/java/com/appoptics/opentelemetry/instrumentation/AoStatementInstrumentation.java
@@ -6,22 +6,17 @@
 package com.appoptics.opentelemetry.instrumentation;
 
 import io.opentelemetry.context.Context;
-import io.opentelemetry.context.Scope;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeTransformer;
-import io.opentelemetry.javaagent.instrumentation.api.CallDepthThreadLocalMap;
+//import io.opentelemetry.javaagent.instrumentation.api.CallDepthThreadLocalMap;
 import net.bytebuddy.asm.Advice;
-import net.bytebuddy.description.method.MethodDescription;
 import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
 import java.sql.Statement;
-import java.util.Map;
 
 import static io.opentelemetry.javaagent.extension.matcher.AgentElementMatchers.implementsInterface;
-import static io.opentelemetry.javaagent.extension.matcher.ClassLoaderMatcher.hasClassesNamed;
-import static io.opentelemetry.javaagent.instrumentation.api.Java8BytecodeBridge.currentContext;
-import static java.util.Collections.singletonMap;
+import static io.opentelemetry.javaagent.extension.matcher.AgentElementMatchers.hasClassesNamed;
 import static net.bytebuddy.matcher.ElementMatchers.*;
 
 /**
@@ -52,9 +47,10 @@ public static class StatementAdvice {
         //@Advice.OnMethodEnter(suppress = Throwable.class)
         @Advice.OnMethodEnter
         public static void onEnter() {
-            if (CallDepthThreadLocalMap.getCallDepth(Statement.class).get() != 1) { //only report back when depth is one to avoid duplications
-                return;
-            }
+            // TODO: CallDepthThreadLocalMap is no longer public. See what we can do
+//            if (CallDepthThreadLocalMap.getCallDepth(Statement.class).get() != 1) { //only report back when depth is one to avoid duplications
+//                return;
+//            }
             AoStatementTracer.writeStackTrace(Context.current());
         }
     }

diff --git a/...ain/java/com/appoptics/opentelemetry/instrumentation/AoHandlerAdapterInstrumentation.java b/...ain/java/com/appoptics/opentelemetry/instrumentation/AoHandlerAdapterInstrumentation.java
@@ -18,7 +18,7 @@
 import javax.servlet.http.HttpServletRequest;
 
 import static io.opentelemetry.javaagent.extension.matcher.AgentElementMatchers.implementsInterface;
-import static io.opentelemetry.javaagent.extension.matcher.ClassLoaderMatcher.hasClassesNamed;
+import static io.opentelemetry.javaagent.extension.matcher.AgentElementMatchers.hasClassesNamed;
 import static net.bytebuddy.matcher.ElementMatchers.*;
 
 /**