Uh oh!
There was an error while loading. Please reload this page.
[fix](Nereids) fix fe folding constant of string functions and add more cases - #45233
Conversation
doris-robot
commented
Dec 10, 2024
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
LiBinfeng-01
commented
Dec 10, 2024
run buildall |
doris-robot
commented
Dec 10, 2024
TPC-H: Total hot run time: 40102 ms |
doris-robot
commented
Dec 10, 2024
TPC-DS: Total hot run time: 198377 ms |
doris-robot
commented
Dec 10, 2024
ClickBench: Total hot run time: 32.22 s |
8e936a9 to
7a9d32bCompareLiBinfeng-01
commented
Dec 11, 2024
run buildall |
doris-robot
commented
Dec 11, 2024
TPC-H: Total hot run time: 40232 ms |
doris-robot
commented
Dec 11, 2024
TPC-DS: Total hot run time: 195402 ms |
doris-robot
commented
Dec 11, 2024
ClickBench: Total hot run time: 32.39 s |
LiBinfeng-01
commented
Dec 12, 2024
run buildall |
doris-robot
commented
Dec 12, 2024
TPC-H: Total hot run time: 39851 ms |
doris-robot
commented
Dec 12, 2024
TPC-DS: Total hot run time: 196138 ms |
doris-robot
commented
Dec 12, 2024
ClickBench: Total hot run time: 32.16 s |
LiBinfeng-01
commented
Dec 12, 2024
run buildall |
1 similar comment
LiBinfeng-01
commented
Dec 13, 2024
run buildall |
doris-robot
commented
Dec 13, 2024
TPC-H: Total hot run time: 39705 ms |
doris-robot
commented
Dec 13, 2024
TPC-DS: Total hot run time: 196387 ms |
doris-robot
commented
Dec 13, 2024
ClickBench: Total hot run time: 32.24 s |
morrySnow
commented
Dec 13, 2024
run p0 |
Uh oh!
There was an error while loading. Please reload this page.
LiBinfeng-01
commented
Dec 17, 2024
run buildall |
| */ | ||
| @ExecFunction(name = "rtrim") | ||
| public static Expression rtrimVarcharVarchar(StringLikeLiteral first, StringLikeLiteral second) { | ||
| public static Expression rtrimVarchar(StringLikeLiteral first, StringLikeLiteral second) { |
There was a problem hiding this comment.
why remove signature in name? and give it a meaningless name?
There was a problem hiding this comment.
because Signature in name is included by StringLikeLiteral, we need this signature before to indicate it is VarcharLiteral or StringLiteral. But now we can use StringlikeLiteral to include them all. Keep one in function to indicate the difference between rtrim and rtrimVarchar
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
8cff18a to
b607a04Comparedoris-robot
commented
Jan 2, 2025
TPC-H: Total hot run time: 32472 ms |
doris-robot
commented
Jan 2, 2025
TPC-DS: Total hot run time: 189907 ms |
doris-robot
commented
Jan 2, 2025
ClickBench: Total hot run time: 30.86 s |
Uh oh!
There was an error while loading. Please reload this page.
LiBinfeng-01
commented
Jan 6, 2025
run buildall |
doris-robot
commented
Jan 6, 2025
TPC-H: Total hot run time: 32871 ms |
doris-robot
commented
Jan 6, 2025
TPC-DS: Total hot run time: 196664 ms |
doris-robot
commented
Jan 6, 2025
ClickBench: Total hot run time: 31.33 s |
PR approved by at least one committer and no changes requested. |
LiBinfeng-01
commented
Jan 6, 2025
run cloud_p0 |
…re cases (apache#45233) Issue Number: apache#44666 Related PR: apache#40441 Problem Summary: - select substring_index('哈哈哈AAA','A', 1); String.split function has second parameter 'limit', which is default zero. When 'limit' is zero, it means it would remove trailing empty strings split of '哈哈哈AAA', which would be '哈哈哈' only. But what we expect is '哈哈哈', '','','' when part function is used by substring index. So we should change splitpart limit to -1 to enable trailing empty character in splitpart list - reorganize fold constant of string functions in fe and add more cases
…re cases (apache#45233) Issue Number: apache#44666 Related PR: apache#40441 Problem Summary: - select substring_index('哈哈哈AAA','A', 1); String.split function has second parameter 'limit', which is default zero. When 'limit' is zero, it means it would remove trailing empty strings split of '哈哈哈AAA', which would be '哈哈哈' only. But what we expect is '哈哈哈', '','','' when part function is used by substring index. So we should change splitpart limit to -1 to enable trailing empty character in splitpart list - reorganize fold constant of string functions in fe and add more cases
…re cases (#45233) (#46525) pick: #45233 Issue Number: #44666 Related PR: #40441 Problem Summary: - select substring_index('哈哈哈AAA','A', 1); String.split function has second parameter 'limit', which is default zero. When 'limit' is zero, it means it would remove trailing empty strings split of '哈哈哈AAA', which would be '哈哈哈' only. But what we expect is '哈哈哈', '','','' when part function is used by substring index. So we should change splitpart limit to -1 to enable trailing empty character in splitpart list - reorganize fold constant of string functions in fe and add more cases --------- Co-authored-by: Mryange <59914473+Mryange@users.noreply.github.com>
…s and add more cases #45233 (#46523) pick: #45233 Issue Number: #44666 Related PR: #40441 Problem Summary: - select substring_index('哈哈哈AAA','A', 1); String.split function has second parameter 'limit', which is default zero. When 'limit' is zero, it means it would remove trailing empty strings split of '哈哈哈AAA', which would be '哈哈哈' only. But what we expect is '哈哈哈', '','','' when part function is used by substring index. So we should change splitpart limit to -1 to enable trailing empty character in splitpart list - reorganize fold constant of string functions in fe and add more cases --------- Co-authored-by: Mryange <59914473+Mryange@users.noreply.github.com>
…e cases apache#45233 (apache#46523) pick: apache#45233 Issue Number: apache#44666 Related PR: apache#40441 Problem Summary: select substring_index('哈哈哈AAA','A', 1); String.split function has second parameter 'limit', which is default zero. When 'limit' is zero, it means it would remove trailing empty strings split of '哈哈哈AAA', which would be '哈哈哈' only. But what we expect is '哈哈哈', '','','' when part function is used by substring index. So we should change splitpart limit to -1 to enable trailing empty character in splitpart list reorganize fold constant of string functions in fe and add more cases
Issue Number: close#44666
Related PR: #40441
Problem Summary:
select substring_index('哈哈哈AAA','A', 1);
String.split function has second parameter 'limit', which is default zero. When 'limit' is zero, it means it would remove trailing
empty strings split of '哈哈哈AAA', which would be '哈哈哈' only. But what we expect is '哈哈哈', '','','' when part function is used by substring index.
So we should change splitpart limit to -1 to enable trailing empty character in splitpart list
reorganize fold constant of string functions in fe and add more cases
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)