Skip to content

Commit

Permalink
[TRIVIAL] Drop auction preprocessing metric (#2969)
Browse files Browse the repository at this point in the history
Since the pre-processing job was
removed(#2960), this metric
doesn't make sense anymore.
  • Loading branch information
squadgazzz authored Sep 11, 2024
1 parent 6c6d982 commit 452377a
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions crates/autopilot/src/run_loop.rs
Original file line number Diff line number Diff line change
Expand Up @@ -180,8 +180,6 @@ impl RunLoop {
let single_run_start = Instant::now();
tracing::info!(?auction_id, "solving");

Metrics::pre_processed(single_run_start.elapsed());

let solutions = self.competition(auction_id, auction).await;
if solutions.is_empty() {
tracing::info!("no solutions for auction");
Expand Down Expand Up @@ -828,10 +826,6 @@ struct Metrics {
/// solved and before sending a `settle` request.
auction_postprocessing_time: prometheus::Histogram,

/// Tracks the time spent in pre-processing before sending a `solve`
/// request.
auction_preprocessing_time: prometheus::Histogram,

/// Tracks the time spent running maintenance. This mostly consists of
/// indexing new events.
#[metric(buckets(0, 0.01, 0.05, 0.1, 0.2, 0.5, 1., 2., 5.))]
Expand Down Expand Up @@ -952,12 +946,6 @@ impl Metrics {
.observe(elapsed.as_secs_f64());
}

fn pre_processed(elapsed: Duration) {
Self::get()
.auction_preprocessing_time
.observe(elapsed.as_secs_f64());
}

fn ran_maintenance(elapsed: Duration) {
Self::get()
.service_maintenance_time
Expand Down

0 comments on commit 452377a

Please sign in to comment.