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

Add percentiles to ping plugin #7344

Closed
fhriley opened this issue Apr 16, 2020 · 3 comments
Closed

Add percentiles to ping plugin #7344

fhriley opened this issue Apr 16, 2020 · 3 comments
Labels
area/ping

Comments

@fhriley
Copy link
Contributor

fhriley commented Apr 16, 2020

Feature Request

When measuring response times, percentiles are typically more useful than averages and standard deviations. Since the ping plugin has a native method and can get all times, percentiles can be calculated.
fhriley pushed a commit to fhriley/telegraf that referenced this issue Apr 16, 2020
Fix influxdata#7344: Add percentiles to the ping plugin
7e9bfaf
fhriley pushed a commit to fhriley/telegraf that referenced this issue Apr 16, 2020
Fix influxdata#7344: Add percentiles to the ping plugin
71d9820
fhriley pushed a commit to fhriley/telegraf that referenced this issue Apr 16, 2020
Fix influxdata#7344: Add percentiles to the ping plugin
fdb5a3b
@sjwang90
Copy link
Contributor

Is there any reason for doing this locally versus during query? Can do a percentile really easily with Flux

@fhriley
Copy link
Contributor Author

fhriley commented Aug 12, 2020

You'd be doing the percentiles over an averaged response time. That's not the same thing.

fhriley pushed a commit to fhriley/telegraf that referenced this issue Dec 1, 2020
@fhriley
Fix influxdata#7344: Add percentiles to the ping plugin
f3ba005
@sjwang90
Copy link
Contributor

Closed in #7345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/ping
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fhriley @sjwang90