From d8f887a595a3ee7c78c16d4513ab86242c859f50 Mon Sep 17 00:00:00 2001 From: Joris Bayer Date: Wed, 8 Jun 2022 15:27:47 +0200 Subject: [PATCH] fix(metrics): Stop logging statsd metric per project key (#1295) It seems that we hit a limit by submitting too many statsd metrics, so let's stop emitting BucketCostPerProjectKey. --- relay-metrics/src/aggregation.rs | 5 ----- relay-metrics/src/statsd.rs | 6 ------ 2 files changed, 11 deletions(-) diff --git a/relay-metrics/src/aggregation.rs b/relay-metrics/src/aggregation.rs index 110b5fc65d..225dd8ddcb 100644 --- a/relay-metrics/src/aggregation.rs +++ b/relay-metrics/src/aggregation.rs @@ -1518,11 +1518,6 @@ impl Aggregator { relay_statsd::metric!( gauge(MetricGauges::BucketsCost) = self.cost_tracker.total_cost as u64 ); - for cost in self.cost_tracker.cost_per_project_key.values() { - relay_statsd::metric!( - histogram(MetricHistograms::BucketsCostPerProjectKey) = *cost as f64 - ); - } let mut buckets = HashMap::>::new(); diff --git a/relay-metrics/src/statsd.rs b/relay-metrics/src/statsd.rs index 92197874b6..d0e7f55e50 100644 --- a/relay-metrics/src/statsd.rs +++ b/relay-metrics/src/statsd.rs @@ -103,11 +103,6 @@ pub enum MetricHistograms { /// - `backdated`: A flag indicating whether the metric was reported within the `initial_delay` /// time period (`false`) or after the initial delay has expired (`true`). BucketsDelay, - - /// The storage cost of metrics buckets stored Relay's metrics aggregator, for a project key. - /// - /// See also [`MetricGauges::BucketsCost`]. - BucketsCostPerProjectKey, } impl HistogramMetric for MetricHistograms { @@ -117,7 +112,6 @@ impl HistogramMetric for MetricHistograms { Self::BucketsFlushedPerProject => "metrics.buckets.flushed_per_project", Self::BucketRelativeSize => "metrics.buckets.relative_bucket_size", Self::BucketsDelay => "metrics.buckets.delay", - Self::BucketsCostPerProjectKey => "metrics.buckets.cost_per_project_key", } } }