-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[feature](nereids)support alter table command #44058
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
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
02da480 to
f0fb46d
Compare
|
run buildall |
2 similar comments
|
run buildall |
|
run buildall |
607fe45 to
515bf28
Compare
|
run buildall |
515bf28 to
09cf7f2
Compare
|
run buildall |
09cf7f2 to
bb10851
Compare
|
run buildall |
bb10851 to
4769394
Compare
|
run buildall |
038e648 to
ab7cdb7
Compare
|
run buildall |
4 similar comments
|
run buildall |
|
run buildall |
|
run buildall |
|
run buildall |
a6bf845 to
1bd2db0
Compare
|
run buildall |
1bd2db0 to
8625ab1
Compare
|
run buildall |
8625ab1 to
96ca3c2
Compare
|
run buildall |
1 similar comment
|
run buildall |
21e086a to
7725236
Compare
|
|
run buildall |
b5f4d6b to
d22014b
Compare
|
run buildall |
TPC-H: Total hot run time: 39997 ms |
TPC-DS: Total hot run time: 190440 ms |
ClickBench: Total hot run time: 32.87 s |
|
run buildall |
TPC-H: Total hot run time: 40180 ms |
TPC-DS: Total hot run time: 196776 ms |
ClickBench: Total hot run time: 32.38 s |
|
run buildall |
TPC-H: Total hot run time: 39864 ms |
TPC-DS: Total hot run time: 197699 ms |
ClickBench: Total hot run time: 32.65 s |
Hastyshell
left a comment
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
|
PR approved by anyone and no changes requested. |
dataroaring
left a comment
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
|
PR approved by at least one committer and no changes requested. |
|
After this pr, dropping column in mv is not allowed |
### What problem does this PR solve? Problem Summary: Since lack of ability of materialized views to modify columns with schema change(with expressions), forbid modifying such columns. This behavior has been established since nereids support alter table command #44058. This PR is for lagecy alter table statement and works for version without nereids' alter table command.
### What problem does this PR solve? Problem Summary: Since lack of ability of materialized views to modify columns with schema change(with expressions), forbid modifying such columns. This behavior has been established since nereids support alter table command apache#44058. This PR is for lagecy alter table statement and works for version without nereids' alter table command.
### What problem does this PR solve? Problem Summary: Since lack of ability of materialized views to modify columns with schema change(with expressions), forbid modifying such columns. This behavior has been established since nereids support alter table command apache#44058. This PR is for lagecy alter table statement and works for version without nereids' alter table command.
support alter table command in nereids
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)