-
Notifications
You must be signed in to change notification settings - Fork 493
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 with join + eval #331
Comments
Kapacitor 0.10.1 (git: master 22b917b) |
@naude-r Thanks, this section of code has been significantly updated, could you try and reproduce the error using the nightly build which can be found here https://influxdata.com/downloads/ |
I have a test case that produces a similar symptom:
Produces this panic:
This is on Kapacitor 0.12. The test case can be found in tests/issue-331 in https://github.com/wildducktheories/influxdata-testcase |
@nathanielc @naude-r I rectified the error in my case by changing my script from:
to:
that is, by adding prefixes for each arm of the join |
…finition time. Assuming the acceptance of PR influxdata#443, some validations previously performed at runtime can now be performed at definition time. The existing validations are moved into the pipeline.JoinNode type. The validations are extended to resolve the panic reported in issue influxdata#331. Signed-off-by: Jon Seymour <jon@wildducktheories.com>
…finition time. Assuming the acceptance of PR influxdata#443, some validations previously performed at runtime can now be performed at definition time. The existing validations are moved into the pipeline.JoinNode type. The validations are extended to resolve the panic reported in issue influxdata#331. Signed-off-by: Jon Seymour <jon@wildducktheories.com>
…ime. Assuming the acceptance of PR #443, some validations previously performed at runtime can now be performed at definition time. The existing validations are moved into the pipeline.JoinNode type. The validations are extended to resolve the panic reported in issue #331. Signed-off-by: Jon Seymour <jon@wildducktheories.com>
* PR #431: Move some join node validations from runtime to definition time. Assuming the acceptance of PR #443, some validations previously performed at runtime can now be performed at definition time. The existing validations are moved into the pipeline.JoinNode type. The validations are extended to resolve the panic reported in issue #331. Signed-off-by: Jon Seymour <jon@wildducktheories.com> * CHANGELOG.md
Exceprt of TICKscript
all other tasks were disabled to ensure a cleaner stacktrace:
The text was updated successfully, but these errors were encountered: