-
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
Panic on groupby #236
Comments
Can you post the panic stack trace? |
The panic is from here, on the default case - https://github.com/influxdata/kapacitor/blob/master/group_by.go |
Perfect thanks, Agreed since it makes it hard to restart Kapacitor I will change the error behavior. |
I think it should be validation on the "wants" and "provides" of each node. If I am connecting window node to group node, on the define request I should get errors the window provides batch and group wants stream. |
Hi,
I got panic while I used groupBy in my tick script.
I don't think this is a good strategy for not implemented features, I think it should be an error that will be logged.
For fixing this I needed for remove my Kapacitor data folder and only then my service got up.
The text was updated successfully, but these errors were encountered: