-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[opt](table-function) Optimize explode with block fast path #60352
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
base: master
Are you sure you want to change the base?
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
53d4f4d to
a75d128
Compare
|
run buildall |
9a164c3 to
ead5f70
Compare
|
run buildall |
TPC-H: Total hot run time: 31816 ms |
ClickBench: Total hot run time: 28.06 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
- Add a block fast path for explode-like table functions to expand contiguous nested ranges without per-row get_value() - Fall back to the original row-wise path when the nested range is non-contiguous or unsupported - Add/extend unit tests to cover fast path and fallback cases Test: ./run-be-ut.sh --run --filter='TableFunctionOperatorTest.*:UnnestTest.*'
ead5f70 to
933e147
Compare
|
run buildall |
1 similar comment
|
run buildall |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
1 similar comment
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
TPC-H: Total hot run time: 31928 ms |
TPC-H: Total hot run time: 32118 ms |
ClickBench: Total hot run time: 28.25 s |
ClickBench: Total hot run time: 28.26 s |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
Test: ./run-be-ut.sh --run --filter='TableFunctionOperatorTest.:UnnestTest.'
What problem does this PR solve?
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)