Update sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/state/DefaultSynchronousMetricStorage.java

Author: fandreuz

sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/state/DefaultSynchronousMetricStorage.java

@@ -137,9 +137,10 @@ public void setEnabled(boolean enabled) {
       AggregatorHolder<T, U> localAggregatorHolder = Objects.requireNonNull(aggregatorHolder.get());
       if (localAggregatorHolder.aggregator == Aggregator.drop()) {
         AggregatorHolder<T, U> newHolder = new AggregatorHolder<>(originalAggregator);
-        previousCollectionAggregatorHandles.clear();
         // If this fails, another thread called `setEnabled` and we can discard the current call
-        aggregatorHolder.compareAndSet(localAggregatorHolder, newHolder);
+        if (aggregatorHolder.compareAndSet(localAggregatorHolder, newHolder)) {
+                previousCollectionAggregatorHandles.clear();
+        }
       }
     } else {
       aggregatorHolder.set(new AggregatorHolder<>(Aggregator.drop()));