better metric name

relay-metrics/src/meta/aggregator.rs

@@ -23,7 +23,7 @@ use crate::{statsd::MetricCounters, MetricResourceIdentifier};
 /// of this happening is small enough to just add it to the storage worst case.
 #[derive(Debug)]
 pub struct MetaAggregator {
-    ///
+    /// All tracked code locations.
     locations: hashbrown::HashMap<Scope, HashSet<Location>>,
 
     /// Maximum tracked locations.
@@ -64,7 +64,7 @@ impl MetaAggregator {
             return None;
         }
 
-        relay_statsd::metric!(counter(MetricCounters::MetaLocationUpdate) += 1);
+        relay_statsd::metric!(counter(MetricCounters::MetaAggregatorUpdate) += 1);
         Some(MetricMeta {
             timestamp: meta.timestamp,
             mapping: send_upstream,

relay-metrics/src/meta/protocol.rs

@@ -47,7 +47,7 @@ pub struct Location {
     pub function: Option<String>,
     /// The line number.
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub lineno: Option<u64>, // TODO nachschauen
+    pub lineno: Option<u64>,
 }
 
 /// A Unix timestamp that is truncated to the start of the day.

relay-metrics/src/statsd.rs

@@ -50,7 +50,7 @@ pub enum MetricCounters {
     BucketsDropped,
     /// Incremented every time the meta aggregator emitted an update that needs to be stored or
     /// sent upstream.
-    MetaLocationUpdate,
+    MetaAggregatorUpdate,
     /// Incremented every time a redis key is updated to store or update metadata.
     MetaRedisUpdate,
 }
@@ -61,7 +61,7 @@ impl CounterMetric for MetricCounters {
             Self::MergeHit => "metrics.buckets.merge.hit",
             Self::MergeMiss => "metrics.buckets.merge.miss",
             Self::BucketsDropped => "metrics.buckets.dropped",
-            Self::MetaLocationUpdate => "metrics.meta.agg.miss",
+            Self::MetaAggregatorUpdate => "metrics.meta.agg.update",
             Self::MetaRedisUpdate => "metrics.meta.redis.update",
         }
     }