Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Change response time histogram bucket sizes #6804

Open
ulope opened this issue Jan 29, 2020 · 0 comments
Open

Change response time histogram bucket sizes #6804

ulope opened this issue Jan 29, 2020 · 0 comments
Labels
A-Metrics metrics, measures, stuff we put in Prometheus T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. z-feature (Deprecated Label) z-p2 (Deprecated Label)

Comments

@ulope
Copy link

ulope commented Jan 29, 2020

Description:

The synapse_http_server_response_time_seconds histogram metric uses the default bucket sizes of .005, .01, .025, .05, .075, .1, .25, .5, .75, 1.0, 2.5, 5.0, 7.5, 10.0, _INF.

Considering that this also contains the sync endpoint which is expected to be in the 30s range quite often (and sometimes more for complex responses) it would be useful to change the bucket sizes to better reflect the actual values.

Another option would be to split out the sync endpoint response time into its own metric.

@richvdh richvdh added z-feature (Deprecated Label) z-p2 (Deprecated Label) labels Jan 30, 2020
@reivilibre reivilibre added T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. A-Metrics metrics, measures, stuff we put in Prometheus labels May 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Metrics metrics, measures, stuff we put in Prometheus T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. z-feature (Deprecated Label) z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

3 participants