-
Couldn't load subscription status.
- Fork 832
feat(query): better script engine with dynamic schema support #18838
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
Conversation
| sender.abort(); | ||
| } | ||
| Some(Ok(block)) => { | ||
| if is_dynamic_schema { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@youngsofun for dynamic schema, please review this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/
Summary
This PR implements a better script engine with enhanced support for dynamic schema handling and improved AST parsing for stored procedures and script execution.
Key improvements:
EXECUTE IMMEDIATEstatements that can return different schemas based on runtime executionIdentifierinstead ofStringfor better type safety and script variable supportdeclare cursor from stmt
declare cursor from resultset, open fetch and close
iterator the cursor:
Major changes:
ExecuteImmediateStmtto accept expressions instead of plain stringsScriptBlockOrStmtenum to handle both script blocks and individual statementsIdentifiertypes for namesTests
Type of change
This change is