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.

Sign up for GitHub

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

Jump to bottom

Compute median and percentiles instead of min/average/max #27

Closed
michaelklishin opened this issue Jul 17, 2017 · 0 comments
Closed

Compute median and percentiles instead of min/average/max #27

michaelklishin opened this issue Jul 17, 2017 · 0 comments
Assignees
@acogoluegnes
Labels
bug enhancement
Milestone
2.0.0

Comments

@michaelklishin
Copy link
Member

michaelklishin commented Jul 17, 2017 •
edited
Loading

min/average/max are not very useful metrics (average is not very stable/prone to swings with extreme values, for instance). 9x percentiles are the industry standard for metrics these days.

I think the following set would be sufficient for PerfTest:

  • min
  • median
  • 75th percentile
  • 95th percentile
  • 99th percentile
@michaelklishin michaelklishin added this to the 1.4.0 milestone Jul 17, 2017
@michaelklishin michaelklishin mentioned this issue Jul 17, 2017
Closed
acogoluegnes added a commit that referenced this issue Jul 17, 2017
@acogoluegnes
Add a flag to print median and percentiles
a181d00 
The -S flag can be used to print median and percentiles.
It's not default to avoid a breaking change. It should be
made the default in 2.0.
The metrics are based on Dropwizard Metrics.

Fixes #27
@acogoluegnes acogoluegnes mentioned this issue Jul 17, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@acogoluegnes acogoluegnes

Labels
bug enhancement
Projects
None yet
Milestone
2.0.0
Development

No branches or pull requests
2 participants
@michaelklishin @acogoluegnes