[feat](function) Add limit parameter support for SPLIT_BY_STRING#60892
[feat](function) Add limit parameter support for SPLIT_BY_STRING#60892floation-cutie wants to merge 3 commits intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
TPC-H: Total hot run time: 28972 ms |
TPC-DS: Total hot run time: 183866 ms |
125e522 to
e89a95b
Compare
|
run buildall |
TPC-H: Total hot run time: 28664 ms |
TPC-DS: Total hot run time: 182736 ms |
FE UT Coverage ReportIncrement line coverage |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
25f35e6 to
5d70879
Compare
|
run buildall |
TPC-H: Total hot run time: 28721 ms |
5d70879 to
1c9eaa9
Compare
TPC-DS: Total hot run time: 182960 ms |
1c9eaa9 to
e9c53c3
Compare
|
run buildall |
TPC-H: Total hot run time: 28535 ms |
TPC-DS: Total hot run time: 182599 ms |
FE UT Coverage ReportIncrement line coverage |
33b32c5 to
ed69d1c
Compare
1a0d91e to
76c1542
Compare
76c1542 to
9e968fe
Compare
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 28942 ms |
TPC-DS: Total hot run time: 183940 ms |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
|
@zclllyybb please review |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
1 similar comment
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
What problem does this PR solve?
Issue Number: close #55788
Related PR: #55788
Problem Summary:
This PR implements the enhancement requested in issue #55788, adding an optional third parameter
limitto theSPLIT_BY_STRINGfunction. Thelimitparameter controls the maximum number of splits to perform, limiting the size of the returned array.Solution
limittoSPLIT_BY_STRING(<str>, <separator>[, <limit>])limitis positive: split from left, at mostlimitsplits, returning at mostlimit + 1elementslimitis negative: split from right, at most|limit|splits from the endlimitis not specified or -1: split all occurrences (backward compatible)function_string.cppandfunction_string.hSplitByString.javaandStringArithmetic.javaExamples:
Release note
[Enhancement] Add optional
limitparameter toSPLIT_BY_STRINGfunction to control maximum number of splits.Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)