-
Notifications
You must be signed in to change notification settings - Fork 200
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
feat: support INSERT INTO SELECT #1536
Conversation
@dracoooooo Hi, it seems there are some CI errors, would you mind take a look? |
Due to #1538, the newly added integration test seems unable to consistently pass in the cluster environment. |
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.
As said in #1538 (comment), sql with multiple tables may fails, so currently we can only enable this features in standalone mode.
In cluster mode, we can do a check to ensure tables belonging to same instance.
For now, you can move this test under local dir, something like integration_tests/cases/env/local/dml/insert_into_select.sql
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
Rationale
Close #557.
Detailed Changes
When generating the insert logical plan, alse generate the select logical plan and store it in the insert plan. Then execute the select logical plan in the insert interpreter, convert the result records into RowGroup and then insert it.
Test Plan
CI