rust-prometheus icon indicating copy to clipboard operation
rust-prometheus copied to clipboard

support summary

Open siddontang opened this issue 9 years ago • 11 comments
trafficstars

siddontang avatar Aug 12 '16 06:08 siddontang


Fabian Reinartz [8:01 AM]  
Also don't worry about about client-side histograms (a.k.a summaries)

[8:01]  
provide a summary type but only do the _count and _sum metrics and skip the quantiles, histograms are superior anyway

xiang90 avatar Aug 12 '16 06:08 xiang90

Thank you very much @xiang90 PTAL @overvenus

ngaut avatar Aug 12 '16 06:08 ngaut

@xiang90 Thanks for reminding

overvenus avatar Aug 15 '16 02:08 overvenus

If you look at the meter in rust-metrics we might get far.

posix4e avatar Aug 31 '16 15:08 posix4e

Thanks @posix4e, we will see it.

siddontang avatar Sep 01 '16 00:09 siddontang

IME, summaries aren't very useful in practice since you can't meaningfully aggregate them across multiple instances of a service. The prometheus docs go into this in some detail, and the go client has deprecated the auto-instrumentation functions for HTTP handlers for this reason (amongst others).

ccmtaylor avatar Sep 13 '16 19:09 ccmtaylor

Thanks @ccmtaylor

The summary has a low priority, but if other clients still support it, we may support it later.

siddontang avatar Sep 14 '16 00:09 siddontang

any news on this?

kaxap avatar Jun 28 '20 20:06 kaxap

Hi, @kaxap currently there is no on-going work for Summary.

breezewish avatar Jun 30 '20 07:06 breezewish

I would personally close this issue summaries are an anti-pattern and when using buckets can be easily derived. https://prometheus.io/docs/practices/histograms/

hdost avatar Dec 09 '20 14:12 hdost

Any news on this?

I try to find a way to integrate Prometheus with AWS CloudWatch, CloudWatch does not support Histogram(ContainerInsights-Prometheus-metrics-conversion), so I try to replace histogram with Summary(https://github.com/starcoinorg/starcoin/issues/1738).

jolestar avatar Oct 13 '21 06:10 jolestar