Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Feat: Support U64 Guage type for Prometheus metrics #30

Closed
roshaans opened this issue Dec 5, 2022 · 0 comments
Closed

Feat: Support U64 Guage type for Prometheus metrics #30

roshaans opened this issue Dec 5, 2022 · 0 comments

Comments

@roshaans
Copy link
Contributor

roshaans commented Dec 5, 2022

We are currently using an IntGauge metric type by rust-prometheus which supports i64 values to store the latest block height seen by the indexer. In its current state, this is fine, but as time goes by and this value gets larger and eventually goes out of bounds, it can cause a runtime exception or give us skewed metrics. We would like to avoid casting the u64 block height into a i64 value and instead use a guage metric type that supports u64 directly.

rust-prometheus currently does not support a Guage type for u64 values, so we have submitted an issue for that.

tikv/rust-prometheus#470

@roshaans roshaans closed this as not planned Won't fix, can't repro, duplicate, stale Dec 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant