From 534066c424779eb7bdd181c65c91a0e625b4ee70 Mon Sep 17 00:00:00 2001 From: Jack Berg <34418638+jack-berg@users.noreply.github.com> Date: Wed, 1 Jul 2026 13:39:29 -0500 Subject: [PATCH] Only set valuesRecorded if its false --- .../sdk/metrics/internal/aggregator/AggregatorHandle.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/aggregator/AggregatorHandle.java b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/aggregator/AggregatorHandle.java index 2115d8dd3c8..454ca707a69 100644 --- a/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/aggregator/AggregatorHandle.java +++ b/sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/aggregator/AggregatorHandle.java @@ -107,7 +107,9 @@ public void recordLong(long value, Attributes attributes, Context context) { throwUnsupportedIfNull(this.longReservoirFactory, UNSUPPORTED_LONG_MESSAGE) .offerLongMeasurement(value, attributes, context); doRecordLong(value); - valuesRecorded = true; + if (!valuesRecorded) { + valuesRecorded = true; + } } /** @@ -122,7 +124,9 @@ public final void recordDouble(double value, Attributes attributes, Context cont throwUnsupportedIfNull(this.doubleReservoirFactory, UNSUPPORTED_DOUBLE_MESSAGE) .offerDoubleMeasurement(value, attributes, context); doRecordDouble(value); - valuesRecorded = true; + if (!valuesRecorded) { + valuesRecorded = true; + } } /**