fix

Author: trask

instrumentation/armeria/armeria-1.3/library/src/main/java/io/opentelemetry/instrumentation/armeria/v1_3/ArmeriaClientTelemetryBuilder.java

@@ -29,6 +29,8 @@ public final class ArmeriaClientTelemetryBuilder {
     ArmeriaInstrumenterBuilderUtil.setClientBuilderExtractor(builder -> builder.builder);
     Experimental.setSetEmitExperimentalClientTelemetry(
         (builder, emit) -> builder.builder.setEmitExperimentalHttpClientMetrics(emit));
+    Experimental.setSetClientPeerService(
+        (builder, peerService) -> builder.builder.setPeerService(peerService));
   }
 
   ArmeriaClientTelemetryBuilder(OpenTelemetry openTelemetry) {
@@ -109,14 +111,6 @@ public ArmeriaClientTelemetryBuilder setSpanNameExtractor(
     return this;
   }
 
-  /**
-   * Can be used via the unstable method {@link
-   * Experimental#setClientPeerService(ArmeriaClientTelemetryBuilder, String)}.
-   */
-  void setPeerService(String peerService) {
-    builder.setPeerService(peerService);
-  }
-
   /**
    * Can be used via the unstable method {@link
    * Experimental#setEmitExperimentalTelemetry(ArmeriaClientTelemetryBuilder, boolean)}.

instrumentation/armeria/armeria-1.3/library/src/main/java/io/opentelemetry/instrumentation/armeria/v1_3/internal/Experimental.java

@@ -15,7 +15,7 @@
  * APIs (or a version of them) may be promoted to the public stable API in the future, but no
  * guarantees are made.
  */
-public class Experimental {
+public final class Experimental {
 
   @Nullable
   private static volatile BiConsumer<ArmeriaClientTelemetryBuilder, Boolean>
@@ -25,20 +25,30 @@ public class Experimental {
   private static volatile BiConsumer<ArmeriaServerTelemetryBuilder, Boolean>
       setEmitExperimentalServerTelemetry;
 
-  public void setEmitExperimentalTelemetry(
+  @Nullable
+  private static volatile BiConsumer<ArmeriaClientTelemetryBuilder, String> setClientPeerService;
+
+  public static void setEmitExperimentalTelemetry(
       ArmeriaClientTelemetryBuilder builder, boolean emitExperimentalTelemetry) {
     if (setEmitExperimentalClientTelemetry != null) {
       setEmitExperimentalClientTelemetry.accept(builder, emitExperimentalTelemetry);
     }
   }
 
-  public void setEmitExperimentalTelemetry(
+  public static void setEmitExperimentalTelemetry(
       ArmeriaServerTelemetryBuilder builder, boolean emitExperimentalTelemetry) {
     if (setEmitExperimentalServerTelemetry != null) {
       setEmitExperimentalServerTelemetry.accept(builder, emitExperimentalTelemetry);
     }
   }
 
+  public static void setClientPeerService(
+      ArmeriaClientTelemetryBuilder builder, String peerService) {
+    if (setClientPeerService != null) {
+      setClientPeerService.accept(builder, peerService);
+    }
+  }
+
   public static void setSetEmitExperimentalClientTelemetry(
       BiConsumer<ArmeriaClientTelemetryBuilder, Boolean> setEmitExperimentalClientTelemetry) {
     Experimental.setEmitExperimentalClientTelemetry = setEmitExperimentalClientTelemetry;
@@ -48,4 +58,11 @@ public static void setSetEmitExperimentalServerTelemetry(
       BiConsumer<ArmeriaServerTelemetryBuilder, Boolean> setEmitExperimentalServerTelemetry) {
     Experimental.setEmitExperimentalServerTelemetry = setEmitExperimentalServerTelemetry;
   }
+
+  public static void setSetClientPeerService(
+      BiConsumer<ArmeriaClientTelemetryBuilder, String> setClientPeerService) {
+    Experimental.setClientPeerService = setClientPeerService;
+  }
+
+  private Experimental() {}
 }