Rename SdkTracerMetrics to SdkTracerInstrumentation (#7930)

Author: anuraaga

sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpanBuilder.java

@@ -205,7 +205,7 @@ public Span startSpan() {
             tracerSharedState.isIdGeneratorSafeToSkipIdValidation());
 
     Runnable recordEndSpanMetrics =
-        tracerSharedState.getTracerMetrics().startSpan(parentSpanContext, samplingDecision);
+        tracerSharedState.getTracerInstrumentation().startSpan(parentSpanContext, samplingDecision);
 
     if (!isRecording(samplingDecision)) {
       return Span.wrap(spanContext);

…elemetry/sdk/trace/SdkTracerMetrics.java …/sdk/trace/SdkTracerInstrumentation.javasdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkTracerMetrics.java renamed to sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkTracerInstrumentation.java sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkTracerMetrics.java renamed to sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkTracerInstrumentation.java

@@ -23,7 +23,7 @@
  * href="https://opentelemetry.io/docs/specs/semconv/otel/sdk-metrics/#span-metrics">semantic
  * conventions</a>.
  */
-final class SdkTracerMetrics {
+final class SdkTracerInstrumentation {
 
   private static final Attributes noParentDrop =
       Attributes.of(
@@ -92,7 +92,7 @@ final class SdkTracerMetrics {
   @Nullable private volatile LongCounter startedSpans;
   @Nullable private volatile LongUpDownCounter liveSpans;
 
-  SdkTracerMetrics(Supplier<MeterProvider> meterProvider) {
+  SdkTracerInstrumentation(Supplier<MeterProvider> meterProvider) {
     this.meterProvider = meterProvider;
   }
 
@@ -105,7 +105,7 @@ Runnable startSpan(SpanContext parentSpanContext, SamplingDecision samplingDecis
       switch (samplingDecision) {
         case DROP:
           startedSpans().add(1, noParentDrop);
-          return SdkTracerMetrics::noop;
+          return SdkTracerInstrumentation::noop;
         case RECORD_ONLY:
           startedSpans().add(1, noParentRecordOnly);
           liveSpans().add(1, recordOnly);
@@ -120,7 +120,7 @@ Runnable startSpan(SpanContext parentSpanContext, SamplingDecision samplingDecis
       switch (samplingDecision) {
         case DROP:
           startedSpans().add(1, remoteParentDrop);
-          return SdkTracerMetrics::noop;
+          return SdkTracerInstrumentation::noop;
         case RECORD_ONLY:
           startedSpans().add(1, remoteParentRecordOnly);
           liveSpans().add(1, recordOnly);
@@ -136,7 +136,7 @@ Runnable startSpan(SpanContext parentSpanContext, SamplingDecision samplingDecis
     switch (samplingDecision) {
       case DROP:
         startedSpans().add(1, localParentDrop);
-        return SdkTracerMetrics::noop;
+        return SdkTracerInstrumentation::noop;
       case RECORD_ONLY:
         startedSpans().add(1, localParentRecordOnly);
         liveSpans().add(1, recordOnly);

sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkTracerProvider.java

@@ -66,7 +66,7 @@ public static SdkTracerProviderBuilder builder() {
             sampler,
             spanProcessors,
             exceptionAttributeResolver,
-            new SdkTracerMetrics(meterProvider));
+            new SdkTracerInstrumentation(meterProvider));
     this.tracerSdkComponentRegistry =
         new ComponentRegistry<>(
             instrumentationScopeInfo ->

sdk/trace/src/main/java/io/opentelemetry/sdk/trace/TracerSharedState.java

@@ -28,7 +28,7 @@ final class TracerSharedState {
   private final Sampler sampler;
   private final SpanProcessor activeSpanProcessor;
   private final ExceptionAttributeResolver exceptionAttributeResolver;
-  private final SdkTracerMetrics tracerMetrics;
+  private final SdkTracerInstrumentation tracerInstrumentation;
 
   @Nullable private volatile CompletableResultCode shutdownResult = null;
 
@@ -40,7 +40,7 @@ final class TracerSharedState {
       Sampler sampler,
       List<SpanProcessor> spanProcessors,
       ExceptionAttributeResolver exceptionAttributeResolver,
-      SdkTracerMetrics tracerMetrics) {
+      SdkTracerInstrumentation tracerInstrumentation) {
     this.clock = clock;
     this.idGenerator = idGenerator;
     this.idGeneratorSafeToSkipIdValidation = idGenerator instanceof RandomIdGenerator;
@@ -49,7 +49,7 @@ final class TracerSharedState {
     this.sampler = sampler;
     this.activeSpanProcessor = SpanProcessor.composite(spanProcessors);
     this.exceptionAttributeResolver = exceptionAttributeResolver;
-    this.tracerMetrics = tracerMetrics;
+    this.tracerInstrumentation = tracerInstrumentation;
   }
 
   Clock getClock() {
@@ -101,8 +101,8 @@ ExceptionAttributeResolver getExceptionAttributesResolver() {
     return exceptionAttributeResolver;
   }
 
-  SdkTracerMetrics getTracerMetrics() {
-    return tracerMetrics;
+  SdkTracerInstrumentation getTracerInstrumentation() {
+    return tracerInstrumentation;
   }
 
   /**