fix: Lint

getsentry · Aug 27, 2024 · 659b92c · 659b92c
1 parent 8ef64a7
commit 659b92c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 13 deletions.
diff --git a/relay-server/src/services/metrics/aggregator.rs b/relay-server/src/services/metrics/aggregator.rs
@@ -9,7 +9,7 @@ use relay_metrics::aggregator::AggregateMetricsError;
 use relay_metrics::{aggregator, Bucket, UnixTimestamp};
 use relay_system::{Controller, FromMessage, Interface, NoResponse, Recipient, Service, Shutdown};
 
-use crate::statsd::{RelayCounters, RelayHistograms, RelayTimers};
+use crate::statsd::{RelayCounters, RelayHistograms};
 
 /// Aggregator for metric buckets.
 ///
@@ -31,17 +31,6 @@ pub enum Aggregator {
  BucketCountInquiry(BucketCountInquiry, relay_system::Sender<usize>),
 }
 
-impl Aggregator {
- /// Returns the name of the message variant.
- pub fn variant(&self) -> &'static str {
- match self {
- Aggregator::MergeBuckets(_) => "MergeBuckets",
- #[cfg(test)]
- Aggregator::BucketCountInquiry(_, _) => "BucketCountInquiry",
- }
- }
-}
-
 impl Interface for Aggregator {}
 
 impl FromMessage<MergeBuckets> for Aggregator {

diff --git a/relay-server/src/services/metrics/router.rs b/relay-server/src/services/metrics/router.rs
@@ -9,7 +9,6 @@ use relay_system::{Addr, NoResponse, Recipient, Service};
 use crate::services::metrics::{
  Aggregator, AggregatorHandle, AggregatorService, FlushBuckets, MergeBuckets,
 };
-use crate::statsd::RelayTimers;
 use crate::utils;
 
 /// Service that routes metrics & metric buckets to the appropriate aggregator.

diff --git a/relay-server/src/statsd.rs b/relay-server/src/statsd.rs
@@ -464,6 +464,7 @@ pub enum RelayTimers {
  /// This metric is tagged with:
  ///
  /// - `message`: The type of message that was processed.
+ #[allow(dead_code)] // TODO: Temporarily disabled for a performance measurement.
  ProcessMessageDuration,
  /// Timing in milliseconds for handling a project cache message.
  ///
@@ -503,11 +504,13 @@ pub enum RelayTimers {
  ///
  /// This metric is tagged with:
  /// - `message`: The type of message that was processed.
+ #[allow(dead_code)] // TODO: Temporarily disabled for a performance measurement.
  AggregatorServiceDuration,
  /// Timing in milliseconds for processing a message in the metric router service.
  ///
  /// This metric is tagged with:
  /// - `message`: The type of message that was processed.
+ #[allow(dead_code)] // TODO: Temporarily disabled for a performance measurement.
  MetricRouterServiceDuration,
  /// Timing in milliseconds for processing a message in the metric store service.
  ///