-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](schema-change) Complete check for string type length change #48607
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) Complete check for string type length change #48607
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
3cf92f3 to
2f08f40
Compare
2f08f40 to
52729b7
Compare
52729b7 to
0790619
Compare
0790619 to
a491cc3
Compare
|
run buildall |
TPC-H: Total hot run time: 31694 ms |
TPC-DS: Total hot run time: 191769 ms |
ClickBench: Total hot run time: 31.06 s |
|
run buildall |
|
run buildall |
TPC-H: Total hot run time: 31603 ms |
TPC-DS: Total hot run time: 190568 ms |
ClickBench: Total hot run time: 30.52 s |
|
run buildall |
|
run buildall |
eldenmoon
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. |
|
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 31504 ms |
amorynan
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
TPC-DS: Total hot run time: 183892 ms |
ClickBench: Total hot run time: 30.88 s |
|
run cloud_p0 |
|
run external |
|
run p0 |
|
run buildall |
TPC-H: Total hot run time: 32407 ms |
TPC-DS: Total hot run time: 191246 ms |
ClickBench: Total hot run time: 30.81 s |
eldenmoon
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
…ache#48607) Related PR: introduce apache#46639 Problem Summary: After the change of apache#46639, shorten length for CHAR type escapes from checking of schema change. Fix in this PR and add some regression test cases to verify it.
…ar length with light schema change (apache#49452) Introduce: apache#48607, apache#46639 Problem Summary: Currently nested types only support light schema change for internal fields, for string types, only varchar can do light schema change.
…ache#48607) ### What problem does this PR solve? Related PR: introduce apache#46639 Problem Summary: After the change of apache#46639, shorten length for CHAR type escapes from checking of schema change. Fix in this PR and add some regression test cases to verify it.
…ar length with light schema change (apache#49452) Introduce: apache#48607, apache#46639 Problem Summary: Currently nested types only support light schema change for internal fields, for string types, only varchar can do light schema change.
What problem does this PR solve?
Related PR: introduce #46639
Problem Summary:
After the change of #46639, shorten length for CHAR type escapes from checking of schema change. Fix in this PR and add some regression test cases to verify it.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)