Skip to content

Commit

Permalink
dex: 🪣 add two larger dex buckets
Browse files Browse the repository at this point in the history
fixes #4464.

this adds two larger buckets to the dex component's histograms.

when our dashboards calculate quantiles, we observed signals that some
operations were taking longer than 100ms. to help obtain more accurate
performance data, we add a 1 second and 10 second bucket.
  • Loading branch information
cratelyn committed May 28, 2024
1 parent fc385e5 commit 807efbd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crates/core/component/dex/src/component/metrics.rs
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ pub fn register_metrics() {
// Prometheus metrics are structured as a Histogram, rather than as a Summary.
// These values are loosely based on the initial Summary output, and may need to be
// updated over time.
pub const DEX_BUCKETS: &[f64; 5] = &[0.00001, 0.0001, 0.001, 0.01, 0.1];
pub const DEX_BUCKETS: &[f64; 5] = &[0.00001, 0.0001, 0.001, 0.01, 0.1, 1, 10];

pub const DEX_PATH_SEARCH_DURATION: &str = "penumbra_dex_path_search_duration_seconds";
pub const DEX_ROUTE_FILL_DURATION: &str = "penumbra_dex_route_fill_duration_seconds";
Expand Down

0 comments on commit 807efbd

Please sign in to comment.