refactor(exporter-metrics-otlp-http): fix eslint warning (#5396)

open-telemetry · Jan 30, 2025 · bb21233 · bb21233
1 parent e265e47
commit bb21233
Showing 1 changed file with 6 additions and 10 deletions.
diff --git a/experimental/packages/opentelemetry-exporter-metrics-otlp-http/src/OTLPMetricExporterBase.ts b/experimental/packages/opentelemetry-exporter-metrics-otlp-http/src/OTLPMetricExporterBase.ts
@@ -111,18 +111,14 @@ function chooseTemporalitySelector(
   return chooseTemporalitySelectorFromEnvironment();
 }
 
+const DEFAULT_AGGREGATION = Object.freeze({
+  type: AggregationType.DEFAULT,
+});
+
 function chooseAggregationSelector(
   config: OTLPMetricExporterOptions | undefined
-) {
-  if (config?.aggregationPreference) {
-    return config.aggregationPreference;
-  } else {
-    return (_instrumentType: any) => {
-      return {
-        type: AggregationType.DEFAULT,
-      };
-    };
-  }
+): AggregationSelector {
+  return config?.aggregationPreference ?? (() => DEFAULT_AGGREGATION);
 }
 
 export class OTLPMetricExporterBase