New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add some metrics #2323
Add some metrics #2323
Conversation
STORE_ENGINE_NUM_SNAPSHOTS_GAUGE.set(n as f64); | ||
} | ||
if let Some(t) = engine.get_property_int(ROCKSDB_OLDEST_SNAPSHOT_TIME) { | ||
let d = time::get_time().sec.checked_sub(t as i64).unwrap_or(0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the time is second?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rest LGTM
/rebuild |
|
||
pub const ENGINE_TICKER_TYPES: &'static [TickerType] = &[ | ||
TickerType::BlockCacheMiss, | ||
TickerType::BlockCacheHit, | ||
TickerType::BlockCacheAdd, | ||
TickerType::BlockCacheAddFailures, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we get the failure reason?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seem can't
LGTM |
b7d2dfe
to
3f9c0b9
Compare
No description provided.