-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[fix](schema-change) Forbid modifying mv related columns #47271
base: master
Are you sure you want to change the base?
[fix](schema-change) Forbid modifying mv related columns #47271
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
run buildall |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 32114 ms
|
TPC-DS: Total hot run time: 186686 ms
|
ClickBench: Total hot run time: 30.36 s
|
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
f2aac10
run p0 |
run buildall |
TPC-H: Total hot run time: 32257 ms
|
TPC-DS: Total hot run time: 184299 ms
|
ClickBench: Total hot run time: 30.47 s
|
run p0 |
73af84f
to
316d6e4
Compare
run buildall |
TPC-H: Total hot run time: 32618 ms
|
TPC-DS: Total hot run time: 184717 ms
|
ClickBench: Total hot run time: 31.27 s
|
run p0 |
…gSiyang2001/doris into forbid-mod-mv-related-column
run buildall |
TPC-H: Total hot run time: 32169 ms
|
TPC-DS: Total hot run time: 190945 ms
|
ClickBench: Total hot run time: 30.72 s
|
run buildall |
PR approved by at least one committer and no changes requested. |
b68ab7c
to
b358486
Compare
run buildall |
PR approved by at least one committer and no changes requested. |
TPC-H: Total hot run time: 32335 ms
|
TPC-DS: Total hot run time: 192148 ms
|
ClickBench: Total hot run time: 30.12 s
|
run p0 |
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.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)