remove reflection

Author: trask

instrumentation/ktor/ktor-2-common/library/src/main/kotlin/io/opentelemetry/instrumentation/ktor/v2_0/common/client/AbstractKtorClientTelemetryBuilder.kt

@@ -13,6 +13,7 @@ import io.opentelemetry.api.common.AttributesBuilder
 import io.opentelemetry.context.Context
 import io.opentelemetry.instrumentation.api.incubator.builder.internal.DefaultHttpClientInstrumenterBuilder
 import io.opentelemetry.instrumentation.api.instrumenter.AttributesExtractor
+import io.opentelemetry.instrumentation.ktor.v2_0.common.client.internal.Experimental
 import io.opentelemetry.instrumentation.ktor.v2_0.common.internal.KtorBuilderUtil
 
 abstract class AbstractKtorClientTelemetryBuilder(
@@ -21,6 +22,7 @@ abstract class AbstractKtorClientTelemetryBuilder(
   companion object {
     init {
       KtorBuilderUtil.clientBuilderExtractor = { it.builder }
+      Experimental.setSetEmitExperimentalTelemetry { builder, emit -> builder.builder.setEmitExperimentalHttpClientMetrics(emit) }
     }
   }
 

instrumentation/ktor/ktor-2-common/library/src/main/kotlin/io/opentelemetry/instrumentation/ktor/v2_0/common/client/AbstractKtorClientTracingBuilder.kt

@@ -160,14 +160,14 @@ abstract class AbstractKtorClientTracingBuilder(
    *
    * @param emitExperimentalHttpClientMetrics `true` if the experimental HTTP client metrics are to be emitted.
    */
-  @Deprecated("Please use method `Experimental.emitExperimentalHttpClientMetrics`")
+  @Deprecated("Please use method `Experimental.emitExperimentalTelemetry`")
   fun setEmitExperimentalHttpClientMetrics(emitExperimentalHttpClientMetrics: Boolean) {
     if (emitExperimentalHttpClientMetrics) {
       emitExperimentalHttpClientMetrics()
     }
   }
 
-  @Deprecated("Please use method `Experimental.emitExperimentalHttpClientMetrics`")
+  @Deprecated("Please use method `Experimental.emitExperimentalTelemetry`")
   fun emitExperimentalHttpClientMetrics() {
     clientBuilder.setEmitExperimentalHttpClientMetrics(true)
   }

instrumentation/ktor/ktor-2-common/library/src/main/kotlin/io/opentelemetry/instrumentation/ktor/v2_0/common/client/internal/Experimental.kt

@@ -1,52 +1,25 @@
-/*
- * Copyright The OpenTelemetry Authors
- * SPDX-License-Identifier: Apache-2.0
- */
-
 package io.opentelemetry.instrumentation.ktor.v2_0.common.client.internal
 
 import io.opentelemetry.instrumentation.ktor.v2_0.common.client.AbstractKtorClientTelemetryBuilder
-import java.lang.reflect.InvocationTargetException
-import java.lang.reflect.Method
-import java.util.logging.Level
-import java.util.logging.Logger
-
-/**
- * This class is internal and experimental. Its APIs are unstable and can change at any time. Its
- * APIs (or a version of them) may be promoted to the public stable API in the future, but no
- * guarantees are made.
- */
-class Experimental {
+import java.util.function.BiConsumer
 
-  companion object {
-    private val logger: Logger = Logger.getLogger(Experimental::class.java.name)
+class Experimental private constructor() {
 
-    private val emitExperimentalTelemetryMethod = getEmitExperimentalTelemetry()
+    companion object {
+        private var setEmitExperimentalTelemetry: BiConsumer<AbstractKtorClientTelemetryBuilder, Boolean>? = null
 
-    fun setEmitExperimentalTelemetry(builder: AbstractKtorClientTelemetryBuilder?, emitExperimentalTelemetry: Boolean) {
-      if (emitExperimentalTelemetryMethod != null) {
-        try {
-          emitExperimentalTelemetryMethod.invoke(builder, emitExperimentalTelemetry)
-        } catch (e: IllegalAccessException) {
-          logger.log(Level.FINE, e.message, e)
-        } catch (e: InvocationTargetException) {
-          logger.log(Level.FINE, e.message, e)
+        fun emitExperimentalTelemetry(
+            builder: AbstractKtorClientTelemetryBuilder
+        ) {
+            if (setEmitExperimentalTelemetry != null) {
+                setEmitExperimentalTelemetry!!.accept(builder, true)
+            }
         }
-      }
-    }
 
-    private fun getEmitExperimentalTelemetry(): Method? {
-      try {
-        val method = AbstractKtorClientTelemetryBuilder::class.java.getDeclaredMethod(
-          "setEmitExperimentalHttpClientMetrics",
-          Boolean::class.javaPrimitiveType
-        )
-        method.setAccessible(true)
-        return method
-      } catch (e: NoSuchMethodException) {
-        logger.log(Level.FINE, e.message, e)
-        return null
-      }
+        fun setSetEmitExperimentalTelemetry(
+            setEmitExperimentalTelemetry: BiConsumer<AbstractKtorClientTelemetryBuilder, Boolean>?
+        ) {
+            Companion.setEmitExperimentalTelemetry = setEmitExperimentalTelemetry
+        }
     }
-  }
 }