Skip to content
This repository was archived by the owner on Jan 14, 2026. It is now read-only.

Add min/max/mean/stddev to histogram#34

Open
serban21 wants to merge 1 commit into
instaclustr:masterfrom
serban21:histogram-mean
Open

Add min/max/mean/stddev to histogram#34
serban21 wants to merge 1 commit into
instaclustr:masterfrom
serban21:histogram-mean

Conversation

@serban21

Copy link
Copy Markdown

Fixes #33

@zegelin

zegelin commented Nov 22, 2019

Copy link
Copy Markdown

This looks pretty good. Thanks.

For adherence to the Prometheus text format exposition format I'd prefer to keep the quantile labels on Summaries as floats, and instead export the min, max, etc. values as separate metrics -- for example metric_max, metric_min, etc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing histogram metrics

2 participants