ExemplarFilter is going to be removed from public API, so don't expos…

…e it
open-telemetry · Feb 19, 2024 · 5c9e395 · 5c9e395
1 parent 8341061
commit 5c9e395
Showing 1 changed file with 14 additions and 20 deletions.
diff --git a/...onfigure/src/main/java/io/opentelemetry/sdk/autoconfigure/MeterProviderConfiguration.java b/...onfigure/src/main/java/io/opentelemetry/sdk/autoconfigure/MeterProviderConfiguration.java
@@ -19,7 +19,6 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Locale;
-import java.util.Optional;
 import java.util.Set;
 import java.util.function.BiFunction;
 import java.util.stream.Collectors;
@@ -34,25 +33,20 @@ static void configureMeterProvider(
           metricExporterCustomizer,
       List<Closeable> closeables) {
 
-    Optional<ExemplarFilter> spiExemplarFilter = spiHelper.loadOptional(ExemplarFilter.class);
-    if (spiExemplarFilter.isPresent()) {
-      SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, spiExemplarFilter.get());
-    } else {
-      // Configure default exemplar filters.
-      String exemplarFilter =
-          config.getString("otel.metrics.exemplar.filter", "trace_based").toLowerCase(Locale.ROOT);
-      switch (exemplarFilter) {
-        case "always_off":
-          SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, ExemplarFilter.alwaysOff());
-          break;
-        case "always_on":
-          SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, ExemplarFilter.alwaysOn());
-          break;
-        case "trace_based":
-        default:
-          SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, ExemplarFilter.traceBased());
-          break;
-      }
+    // Configure default exemplar filters.
+    String exemplarFilter =
+        config.getString("otel.metrics.exemplar.filter", "trace_based").toLowerCase(Locale.ROOT);
+    switch (exemplarFilter) {
+      case "always_off":
+        SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, ExemplarFilter.alwaysOff());
+        break;
+      case "always_on":
+        SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, ExemplarFilter.alwaysOn());
+        break;
+      case "trace_based":
+      default:
+        SdkMeterProviderUtil.setExemplarFilter(meterProviderBuilder, ExemplarFilter.traceBased());
+        break;
     }
 
     int cardinalityLimit =