remove unused CompletableFutureWrapper.java

Author: mznet

instrumentation/opensearch/opensearch-java-3.0/javaagent/build.gradle.kts

@@ -23,7 +23,6 @@ dependencies {
   testImplementation(project(":instrumentation:opensearch:opensearch-rest-common:testing"))
   testInstrumentation(project(":instrumentation:apache-httpclient:apache-httpclient-5.0:javaagent"))
 
-  // AWS SDK 2 dependencies for AwsSdk2Transport
   testImplementation("software.amazon.awssdk:auth:2.22.0")
   testImplementation("software.amazon.awssdk:identity-spi:2.22.0")
   testImplementation("software.amazon.awssdk:apache-client:2.22.0")

instrumentation/opensearch/opensearch-java-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opensearch/java/v3_0/CompletableFutureWrapper.java

@@ -19,6 +19,7 @@ public OpenSearchJavaResponseHandler(Context context, OpenSearchJavaRequest otel
 
   @Override
   public void accept(Object response, Throwable error) {
+    // In OpenSearch responses, there's no information available, so the span is closed with null.
     OpenSearchJavaSingletons.instrumenter().end(context, otelRequest, null, error);
   }
 }

instrumentation/opensearch/opensearch-java-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opensearch/java/v3_0/OpenSearchJavaResponseHandler.java

@@ -42,8 +42,7 @@
 import software.amazon.awssdk.regions.Region;
 import software.amazon.awssdk.utils.AttributeMap;
 
-@SuppressWarnings(
-    "deprecation") // AwsSdk2Transport is the correct way for OpenSearch Java 3.0 with AWS SDK 2
+@SuppressWarnings("deprecation")
 public class OpenSearchJavaAwsSdk2TransportTest extends AbstractOpenSearchJavaTest {
 
   protected static final MockWebServerExtension server = new MockWebServerExtension();