chore: remove useless code in CompatibilityChecker.java

Change-Id: I6eb6e43bb8b579f20a3341de11412e83c853e860

Author: 123liuziming

instrumentation/lettuce/lettuce-5.1/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/lettuce/v5_1/CompatibilityChecker.java

@@ -5,12 +5,9 @@
 
 package io.opentelemetry.javaagent.instrumentation.lettuce.v5_1;
 
-import io.lettuce.core.tracing.TraceContext;
-import io.lettuce.core.tracing.TraceContextProvider;
 import io.lettuce.core.tracing.Tracing;
 import java.util.concurrent.locks.Lock;
 import java.util.concurrent.locks.ReentrantLock;
-import reactor.core.publisher.Mono;
 
 public class CompatibilityChecker {
 
@@ -20,26 +17,14 @@ private CompatibilityChecker() {}
   private static final Lock lock = new ReentrantLock();
 
   // related to https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/10997
-  // if users are using  incompatible versions of reactor-core and lettuce
+  // if users are using incompatible versions of reactor-core and lettuce
   // then just disable the instrumentation
   public static boolean checkCompatible() {
     lock.lock();
     try {
       if (isCompatible != null) {
         return isCompatible;
       }
-      Tracing.withTraceContextProvider(
-          new TraceContextProvider() {
-            @Override
-            public TraceContext getTraceContext() {
-              return null;
-            }
-
-            @Override
-            public Mono<TraceContext> getTraceContextLater() {
-              return TraceContextProvider.super.getTraceContextLater();
-            }
-          });
       Tracing.getContext();
       isCompatible = true;
     } catch (Throwable t) {

instrumentation/lettuce/lettuce-5.1/javaagent/src/testCompatibility/java/io/opentelemetry/javaagent/instrumentation/lettuce/v5_1/LettuceCompatibilityTest.java

@@ -9,6 +9,7 @@
 
 import io.lettuce.core.RedisClient;
 import io.lettuce.core.api.sync.RedisCommands;
+import io.opentelemetry.api.trace.SpanKind;
 import io.opentelemetry.instrumentation.lettuce.v5_1.AbstractLettuceClientTest;
 import io.opentelemetry.instrumentation.testing.junit.AgentInstrumentationExtension;
 import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension;
@@ -60,11 +61,14 @@ static void cleanUp() {
   }
 
   @Test
-  void testEmptyTrace() {
-    String res = syncCommands.set("TESTSETKEY", "TESTSETVAL");
+  void testInstrumentationDisabled() {
+    String res =
+        testing().runWithSpan("parent", () -> syncCommands.set("TESTSETKEY", "TESTSETVAL"));
     assertThat(res).isEqualTo("OK");
-
-    assertThat(testing.spans().isEmpty()).isTrue();
-    assertThat(testing.metrics().isEmpty()).isTrue();
+    testing()
+        .waitAndAssertTraces(
+            trace ->
+                trace.hasSpansSatisfyingExactly(
+                    span -> span.hasName("parent").hasKind(SpanKind.INTERNAL).hasNoParent()));
   }
 }