-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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](sql) Fix error result for column tosql #44997
Conversation
run buildall |
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
TPC-H: Total hot run time: 40297 ms
|
d2d8509
to
9101839
Compare
run buildall |
9101839
to
b34ce67
Compare
run buildall |
TPC-H: Total hot run time: 39950 ms
|
TPC-DS: Total hot run time: 198568 ms
|
ClickBench: Total hot run time: 33.66 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.
same as #44958 it is better to escape value
This PR aims to fix |
@wyxxxcat please add some unit tests. |
You can refer https://github.com/apache/doris/pull/41205/files |
ae79a75
to
7328f74
Compare
run buildall |
TPC-H: Total hot run time: 39836 ms
|
TPC-DS: Total hot run time: 197010 ms
|
ClickBench: Total hot run time: 32.84 s
|
7328f74
to
298c2d1
Compare
run buildall |
TPC-H: Total hot run time: 39826 ms
|
TPC-DS: Total hot run time: 197170 ms
|
ClickBench: Total hot run time: 32.65 s
|
run p0 |
PR approved by anyone and no changes requested. |
please update fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/info/DefaultValue.java for nereids, too |
298c2d1
to
39a8004
Compare
run buildall |
TPC-H: Total hot run time: 40125 ms
|
TPC-DS: Total hot run time: 195892 ms
|
ClickBench: Total hot run time: 32.32 s
|
PR approved by at least one committer and no changes requested. |
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
Cherry-picked from #44997 Co-authored-by: Uniqueyou <[email protected]>
What problem does this PR solve?
related : selectdb/ccr-syncer#293
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)