Skip to content
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

Make percentiles for dogstatsd configurable #165

Closed
alq666 opened this issue Aug 28, 2012 · 2 comments
Closed

Make percentiles for dogstatsd configurable #165

alq666 opened this issue Aug 28, 2012 · 2 comments
Labels

Comments

@alq666
Copy link
Member

alq666 commented Aug 28, 2012

  1. etsy's statsd only computes the 90th by default but allows configurable percentiles
  2. our current 50, 75, 85, 95, 99 is probably too many by default
  3. our initial average was the median; it's deployed widely

My vote goes for default metrics goes to:

  • max
  • count
  • avg
  • median
  • 95% (2 sigmas of normal distro), most people currently use this one.

min is of unclear usefulness so far. If it's measuring time, min always ends up near 0.

5 metrics instead of 8 with average and median

alq666 added a commit that referenced this issue Aug 28, 2012
@olidb2
Copy link
Member

olidb2 commented Aug 29, 2012

Note for future reference: if we make it configurable, it would be good to
have the front end "know" about it or have a way to guess it, so we can
automatically generate min + med + "top" graphs.

More flexibility/configurability => more complicated for the front-end to
automatically show the right thing.

On Tue, Aug 28, 2012 at 12:21 AM, Alexis Lê-Quôc
notifications@github.comwrote:

  1. etsy's statsd only computes the 90th by default but allows
    configurable percentiles
  2. our current 50, 75, 85, 95, 99 is probably too many by default
  3. our initial average was the median; it's deployed widely

My vote goes for:

  • max
  • count
  • avg
  • median
  • 95% (2 sigmas of normal distro)

5 metrics instead of 8 with average and median


Reply to this email directly or view it on GitHubhttps://github.com//issues/165.

@alq666 alq666 closed this as completed Dec 23, 2013
@alq666
Copy link
Member Author

alq666 commented Dec 23, 2013

Already implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants