Skip to content

Commit 54d4b70

Browse files
committed
remove sample rate
1 parent 325d132 commit 54d4b70

File tree

3 files changed

+3
-17
lines changed

3 files changed

+3
-17
lines changed

rust_snuba/src/config.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ pub struct EnvConfig {
108108
pub lower_retention_days: u16,
109109
pub valid_retention_days: HashSet<u16>,
110110
pub record_cogs: bool,
111-
pub ddm_metrics_sample_rate: f64,
112111
pub project_stacktrace_blacklist: Vec<u64>,
113112
}
114113

@@ -122,7 +121,6 @@ impl Default for EnvConfig {
122121
lower_retention_days: 30,
123122
valid_retention_days: [30, 60, 90].iter().cloned().collect(),
124123
record_cogs: false,
125-
ddm_metrics_sample_rate: 0.0,
126124
project_stacktrace_blacklist: Vec::new(),
127125
}
128126
}

rust_snuba/src/consumer.rs

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -179,13 +179,7 @@ pub fn consumer_v2_impl(
179179
set_global_tag("storage".to_owned(), storage_name);
180180
set_global_tag("consumer_group".to_owned(), consumer_group.to_owned());
181181

182-
metrics::init(StatsDBackend::new(
183-
&host,
184-
port,
185-
"snuba.consumer",
186-
env_config.ddm_metrics_sample_rate,
187-
))
188-
.unwrap();
182+
metrics::init(StatsDBackend::new(&host, port, "snuba.consumer")).unwrap();
189183
}
190184

191185
if !use_rust_processor {
@@ -414,13 +408,7 @@ pub fn consumer_impl(
414408
set_global_tag("storage".to_owned(), storage_name);
415409
set_global_tag("consumer_group".to_owned(), consumer_group.to_owned());
416410

417-
metrics::init(StatsDBackend::new(
418-
&host,
419-
port,
420-
"snuba.consumer",
421-
env_config.ddm_metrics_sample_rate,
422-
))
423-
.unwrap();
411+
metrics::init(StatsDBackend::new(&host, port, "snuba.consumer")).unwrap();
424412
}
425413

426414
if !use_rust_processor {

rust_snuba/src/metrics/statsd.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ impl MetricSink for Wrapper {
2828
}
2929

3030
impl StatsDBackend {
31-
pub fn new(host: &str, port: u16, prefix: &str, _ddm_metrics_sample_rate: f64) -> Self {
31+
pub fn new(host: &str, port: u16, prefix: &str) -> Self {
3232
let upstream_addr = format!("{}:{}", host, port);
3333
let aggregator_sink = StatsdProxyMetricSink::new(move || {
3434
let upstream = Upstream::new(upstream_addr.clone()).unwrap();

0 commit comments

Comments
 (0)