chore: Add info on set local aggregator behaviour (#3276)

philipphofmann · web-flow · commit 5f908cb88463 · 2024-03-21T17:18:09.000+01:00
diff --git a/sentry/src/main/java/io/sentry/MetricsAggregator.java b/sentry/src/main/java/io/sentry/MetricsAggregator.java
@@ -219,6 +219,9 @@ private void add(
       }
       totalBucketsWeight.addAndGet(addedWeight);
     }
+
+    // For sets, we only record that a value has been added to the set but not which one.
+    // See develop docs: https://develop.sentry.dev/sdk/metrics/#sets
     if (localMetricsAggregator != null) {
       final double localValue = type == MetricType.Set ? addedWeight : value;
       localMetricsAggregator.add(metricKey, type, key, localValue, unit, tags, timestampMs);

Original file line number	Diff line number	Diff line change
`@@ -219,6 +219,9 @@ private void add(`
`219`	`219`	`}`
`220`	`220`	`totalBucketsWeight.addAndGet(addedWeight);`
`221`	`221`	`}`
	`222`	`+`
	`223`	`+ // For sets, we only record that a value has been added to the set but not which one.`
	`224`	`+ // See develop docs: https://develop.sentry.dev/sdk/metrics/#sets`
`222`	`225`	`if (localMetricsAggregator != null) {`
`223`	`226`	`final double localValue = type == MetricType.Set ? addedWeight : value;`
`224`	`227`	`localMetricsAggregator.add(metricKey, type, key, localValue, unit, tags, timestampMs);`