-
Notifications
You must be signed in to change notification settings - Fork 171
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
Specify how to handle missing time series in metrics #18
Labels
Comments
By not expose do you mean drop? We should definitely drop incomplete events I'd say :) |
By not expose I mean exclude that line from the output. This is an metrics-based standard, there are no events. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We know Summarys will permit missing quantiles.
We should also specify what happens if a Summary is missing _count/_sum or if a Histogram is missing _sum as that can happen with some other monitoring systems. I believe we should not expose them. It should also be made clear that NaN should only be used where it makes mathematical sense, not as a general "missing" signal.
The text was updated successfully, but these errors were encountered: