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

Remove RUN SCRIPT from KSQL syntax #2433

Closed
big-andy-coates opened this issue Feb 12, 2019 · 0 comments · Fixed by #4116
Closed

Remove RUN SCRIPT from KSQL syntax #2433

big-andy-coates opened this issue Feb 12, 2019 · 0 comments · Fixed by #4116
Assignees
Labels
Milestone

Comments

@big-andy-coates
Copy link
Contributor

Issue #2179 saw the RUN SCRIPT functionality being dropped from the rest API and being replaced by functionality within the CLI to pass the script contents in the request body.

However, RUN SCRIPT remains in the KSQL syntax. This issue is here to track the removal of it from the syntax on the next major version bump.

This will mean users of the RESTful API will receive a parser error should they submit a RUN SCRIPT request, rather than the current message explaining its no longer supported. (Note, the CLI still supports RUN SCRIPT).

To support RUN SCRIPT statements already in the command queue we may chose to add custom handling - or make a breaking change. TBD.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants