fix stuff and order

Author: fandreuz

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

@@ -145,12 +145,6 @@ AsynchronousMetricStorage<T, U> create(
         registeredView.getCardinalityLimit());
   }
 
-  @Override
-  public void setEnabled(boolean enabled) {
-    this.aggregatorHolder =
-        new AggregatorHolder<>(enabled ? originalAggregator : Aggregator.drop());
-  }
-
   /** Record callback measurement from {@link ObservableLongMeasurement}. */
   void record(Attributes attributes, long value) {
     attributes = validateAndProcessAttributes(attributes);
@@ -321,12 +315,19 @@ private Collection<T> collectWithCumulativeAggregationTemporality() {
     return currentPoints;
   }
 
+  @Override
+  public void setEnabled(boolean enabled) {
+    this.aggregatorHolder =
+        new AggregatorHolder<>(enabled ? originalAggregator : Aggregator.drop());
+  }
+
   @Override
   public boolean isEmpty() {
     return aggregatorHolder.aggregator == Aggregator.drop();
   }
 
   private static final class AggregatorHolder<T extends PointData, U extends ExemplarData> {
+
     private final Aggregator<T, U> aggregator;
     private final ObjectPool<AggregatorHandle<T, U>> reusableHandlesPool;
     private final BiConsumer<Attributes, AggregatorHandle<T, U>> handleReleaser;

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

@@ -94,13 +94,8 @@ public final class DefaultSynchronousMetricStorage<T extends PointData, U extend
     this.memoryMode = registeredReader.getReader().getMemoryMode();
   }
 
-  @Override
-  public void setEnabled(boolean enabled) {
-    this.aggregatorHolder.set(
-        new AggregatorHolder<>(enabled ? originalAggregator : Aggregator.drop()));
-  }
-
   // Visible for testing
+
   Queue<AggregatorHandle<T, U>> getAggregatorHandlePool() {
     return aggregatorHandlePool;
   }
@@ -137,9 +132,15 @@ public void recordDouble(double value, Attributes attributes, Context context) {
     }
   }
 
+  @Override
+  public void setEnabled(boolean enabled) {
+    this.aggregatorHolder.set(
+        new AggregatorHolder<>(enabled ? originalAggregator : Aggregator.drop()));
+  }
+
   @Override
   public boolean isEnabled() {
-    return true;
+    return this.aggregatorHolder.get().aggregator == Aggregator.drop();
   }
 
   /**