Move method

Author: anuraaga

sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/SdkMeterProvider.java

@@ -29,6 +29,8 @@
 import io.opentelemetry.sdk.metrics.internal.view.ViewRegistry;
 import io.opentelemetry.sdk.resources.Resource;
 import java.io.Closeable;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
@@ -99,7 +101,7 @@ public static SdkMeterProviderBuilder builder() {
       reader.register(new SdkCollectionRegistration(readerMetricProducers, sharedState));
       registeredReader.setLastCollectEpochNanos(startEpochNanos);
       if (reader instanceof PeriodicMetricReader) {
-        SdkMeterProviderUtil.setMeterProvider((PeriodicMetricReader) reader, this);
+        setReaderMeterProvider((PeriodicMetricReader) reader, this);
       }
     }
   }
@@ -198,6 +200,18 @@ public String toString() {
         + "}";
   }
 
+  private static void setReaderMeterProvider(
+      PeriodicMetricReader metricReader, SdkMeterProvider meterProvider) {
+    try {
+      Method method =
+          PeriodicMetricReader.class.getDeclaredMethod("setMeterProvider", MeterProvider.class);
+      method.setAccessible(true);
+      method.invoke(metricReader, meterProvider);
+    } catch (NoSuchMethodException | IllegalAccessException | InvocationTargetException e) {
+      throw new IllegalStateException("Error calling setMeterProvider on PeriodicMetricReader", e);
+    }
+  }
+
   /** Helper class to expose registered metric exports. */
   private static class LeasedMetricProducer implements MetricProducer {
 

sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/SdkMeterProviderUtil.java

@@ -5,13 +5,11 @@
 
 package io.opentelemetry.sdk.metrics.internal;
 
-import io.opentelemetry.api.metrics.MeterProvider;
 import io.opentelemetry.sdk.common.InstrumentationScopeInfo;
 import io.opentelemetry.sdk.common.internal.ScopeConfigurator;
 import io.opentelemetry.sdk.metrics.SdkMeterProvider;
 import io.opentelemetry.sdk.metrics.SdkMeterProviderBuilder;
 import io.opentelemetry.sdk.metrics.ViewBuilder;
-import io.opentelemetry.sdk.metrics.export.PeriodicMetricReader;
 import io.opentelemetry.sdk.metrics.internal.view.AttributesProcessor;
 import io.opentelemetry.sdk.metrics.internal.view.StringPredicates;
 import java.lang.reflect.InvocationTargetException;
@@ -78,19 +76,6 @@ public static SdkMeterProviderBuilder addMeterConfiguratorCondition(
     return sdkMeterProviderBuilder;
   }
 
-  /** Reflectively sets the meter provider for a PeriodicMetricReader to export metrics to. */
-  public static void setMeterProvider(
-      PeriodicMetricReader metricReader, SdkMeterProvider meterProvider) {
-    try {
-      Method method =
-          PeriodicMetricReader.class.getDeclaredMethod("setMeterProvider", MeterProvider.class);
-      method.setAccessible(true);
-      method.invoke(metricReader, meterProvider);
-    } catch (NoSuchMethodException | IllegalAccessException | InvocationTargetException e) {
-      throw new IllegalStateException("Error calling setMeterProvider on PeriodicMetricReader", e);
-    }
-  }
-
   /**
    * Reflectively add an {@link AttributesProcessor} to the {@link ViewBuilder} which appends
    * key-values from baggage to all measurements.