Skip to content

Commit

Permalink
chore: add one forgotten metric (#27406)
Browse files Browse the repository at this point in the history
  • Loading branch information
daibhin authored Jan 10, 2025
1 parent 468f340 commit 2ec15d9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
4 changes: 3 additions & 1 deletion rust/cymbal/src/frames/resolver.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ use sqlx::PgPool;
use crate::{
config::Config,
error::UnhandledError,
metric_consts::{FRAME_CACHE_HITS, FRAME_CACHE_MISSES, FRAME_DB_HITS},
metric_consts::{FRAME_CACHE_HITS, FRAME_CACHE_MISSES, FRAME_DB_HITS, FRAME_DB_MISSES},
symbol_store::{saving::SymbolSetRecord, Catalog},
};

Expand Down Expand Up @@ -49,6 +49,8 @@ impl Resolver {
return Ok(result.contents);
}

metrics::counter!(FRAME_DB_MISSES).increment(1);

let resolved = frame.resolve(team_id, catalog).await?;

let set = if let Some(set_ref) = frame.symbol_set_ref() {
Expand Down
1 change: 1 addition & 0 deletions rust/cymbal/src/metric_consts.rs
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ pub const FRAME_RESOLVED: &str = "cymbal_frame_resolved";
pub const FRAME_CACHE_HITS: &str = "cymbal_frame_cache_hits";
pub const FRAME_CACHE_MISSES: &str = "cymbal_frame_cache_misses";
pub const FRAME_DB_HITS: &str = "cymbal_frame_db_hits";
pub const FRAME_DB_MISSES: &str = "cymbal_frame_db_misses";
pub const FRAME_NOT_RESOLVED: &str = "cymbal_frame_not_resolved";
pub const S3_FETCH: &str = "cymbal_s3_fetch";
pub const S3_PUT: &str = "cymbal_s3_put";
Expand Down

0 comments on commit 2ec15d9

Please sign in to comment.