remove unused OpenSearchJavaResponse.java

Author: mznet

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

@@ -5,14 +5,12 @@
 
 package io.opentelemetry.javaagent.instrumentation.opensearch.java.v3_0;
 
-import static io.opentelemetry.instrumentation.api.incubator.semconv.db.internal.DbResponseStatusUtil.dbResponseStatusCode;
-
 import io.opentelemetry.instrumentation.api.incubator.semconv.db.DbClientAttributesGetter;
 import io.opentelemetry.semconv.incubating.DbIncubatingAttributes;
 import javax.annotation.Nullable;
 
 final class OpenSearchJavaAttributesGetter
-    implements DbClientAttributesGetter<OpenSearchJavaRequest, OpenSearchJavaResponse> {
+    implements DbClientAttributesGetter<OpenSearchJavaRequest, Void> {
 
   @SuppressWarnings("deprecation") // using deprecated DbSystemIncubatingValues
   @Override
@@ -54,8 +52,7 @@ public String getDbOperationName(OpenSearchJavaRequest request) {
 
   @Nullable
   @Override
-  public String getResponseStatus(
-      @Nullable OpenSearchJavaResponse response, @Nullable Throwable error) {
-    return response != null ? dbResponseStatusCode(response.getStatusCode()) : null;
+  public String getResponseStatus(@Nullable Void response, @Nullable Throwable error) {
+    return null; // Response status is handled by HTTP instrumentation
   }
 }

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

@@ -14,11 +14,10 @@
 
 public final class OpenSearchJavaInstrumenterFactory {
 
-  public static Instrumenter<OpenSearchJavaRequest, OpenSearchJavaResponse> create(
-      String instrumentationName) {
+  public static Instrumenter<OpenSearchJavaRequest, Void> create(String instrumentationName) {
     OpenSearchJavaAttributesGetter dbClientAttributesGetter = new OpenSearchJavaAttributesGetter();
 
-    return Instrumenter.<OpenSearchJavaRequest, OpenSearchJavaResponse>builder(
+    return Instrumenter.<OpenSearchJavaRequest, Void>builder(
             GlobalOpenTelemetry.get(),
             instrumentationName,
             DbClientSpanNameExtractor.create(dbClientAttributesGetter))

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

@@ -19,7 +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.
+    // OpenSearch responses don't provide response information, 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/OpenSearchJavaResponse.java

@@ -8,10 +8,10 @@
 import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter;
 
 public final class OpenSearchJavaSingletons {
-  private static final Instrumenter<OpenSearchJavaRequest, OpenSearchJavaResponse> INSTRUMENTER =
+  private static final Instrumenter<OpenSearchJavaRequest, Void> INSTRUMENTER =
       OpenSearchJavaInstrumenterFactory.create("io.opentelemetry.opensearch-java-3.0");
 
-  public static Instrumenter<OpenSearchJavaRequest, OpenSearchJavaResponse> instrumenter() {
+  public static Instrumenter<OpenSearchJavaRequest, Void> instrumenter() {
     return INSTRUMENTER;
   }