-
Notifications
You must be signed in to change notification settings - Fork 490
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
[Feature Request] Adding the .quiet() property to all node types and making it more configurable #1561
Comments
@nathanielc Can we scope this for effort / feasibility? |
Adding a Allowing the suppression of specific errors using a lambda expression is a non-starter. Lambda expressions are tailored to processing the point data, and while it might be possible to re-purpose them, I think it would be a confusing user experience. Another way to solve this would be to have the |
Yeah. Having a .quiet() propety and some parameter to adjust the verbosity (like a list of errors) would be sufficient. Fair point about the lambda expressions. |
A trivial example: We process a measurement where a certain field is not always defined. We want to supress the error that results in a node failing to process that field if it is undefined. We could start adding default nodes but that can become really convoluted. Just having kapacitor drop the problematic line to the floor quietly would be the preferred option. |
The eval node has a .quiet() property which is very useful for supressing errors and warnings that we don't necessarily care about. It would be great if more, or even all, node types would offer this functionality.
Ideally I'd like even with more control over which log level messages get supressed and possibly a lambda expression to supress specific errors. For example, we might want DerivativeNode to supress just the "0 time difference" errors but nothing else.
The text was updated successfully, but these errors were encountered: