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

meter should expose current rate per second #53

Open
mseld opened this issue Feb 24, 2018 · 1 comment
Open

meter should expose current rate per second #53

mseld opened this issue Feb 24, 2018 · 1 comment
Assignees

Comments

@mseld
Copy link

mseld commented Feb 24, 2018

meter should expose current rate per second

{
    "meter": {
        "type": "meter",
        "count": 60,
        "currentRate": 7, // expose 
        "m1": 0.1968129595640299,
        "m5": 0.1356642465884818,
        "m15": 0.057142134744181075,
        "mean": 0.3178959526546961,
        "unit": "seconds"
    }
}
@mseld mseld changed the title meter should return current rate per second meter should expose current rate per second Feb 24, 2018
@tolbertam
Copy link
Collaborator

This may be possible, although I think the utility of it may be limited. The m1/m5/m15 metrics record rate per second over those time period. Rate over the last second is compelling, but because of #37, it's not a good idea as it would require configuring an interval to run every 1 second. If we fix that, this may be possible although I'm not sure how well ExponentiallyWeightedMovingAverage works over short time periods. I'll evaluate as part of #37.

@tolbertam tolbertam self-assigned this Oct 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants