Uh oh!
There was an error while loading. Please reload this page.
[enchement](orc) improve the read amplification problem caused by orc tiny stripe optimization. - #50675
[enchement](orc) improve the read amplification problem caused by orc tiny stripe optimization. #50675hubgeter wants to merge 1 commit into
Conversation
Thearas
commented
May 7, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
hubgeter
commented
May 7, 2025
run buildall |
2801c8d to
3befcf5Comparehubgeter
commented
May 16, 2025
run buildall |
doris-robot
commented
May 16, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
doris-robot
commented
May 16, 2025
TPC-H: Total hot run time: 33873 ms |
doris-robot
commented
May 16, 2025
TPC-DS: Total hot run time: 193233 ms |
doris-robot
commented
May 16, 2025
ClickBench: Total hot run time: 29.83 s |
doris-robot
commented
May 16, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
May 16, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
| range_finder); | ||
| //The first stripe in the read file is selected to calculate an approximate | ||
| // read amplification factor using the tiny stripe optimization. | ||
| std::vector<bool> selectedColumns; |
There was a problem hiding this comment.
| std::vector<bool> selectedColumns; | |
| std::vector<bool> selected_columns; |
3befcf5 to
c3d4c9bComparehubgeter
commented
May 25, 2025
run buildall |
hello-stephen
commented
May 25, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
doris-robot
commented
May 25, 2025
TPC-H: Total hot run time: 33616 ms |
doris-robot
commented
May 25, 2025
TPC-DS: Total hot run time: 192540 ms |
doris-robot
commented
May 25, 2025
ClickBench: Total hot run time: 29.62 s |
hubgeter
commented
May 28, 2025
run buildall |
hello-stephen
commented
May 28, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
doris-robot
commented
May 28, 2025
TPC-H: Total hot run time: 33939 ms |
doris-robot
commented
May 28, 2025
TPC-DS: Total hot run time: 192989 ms |
doris-robot
commented
May 28, 2025
ClickBench: Total hot run time: 29.71 s |
hubgeter
commented
May 28, 2025
run buildall |
hello-stephen
commented
May 28, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
doris-robot
commented
May 28, 2025
TPC-H: Total hot run time: 33607 ms |
doris-robot
commented
May 28, 2025
TPC-DS: Total hot run time: 192368 ms |
doris-robot
commented
May 28, 2025
ClickBench: Total hot run time: 29.7 s |
… tiny stripe optimization.
4516147 to
41221b8Comparehubgeter
commented
May 30, 2025
run buildall |
hello-stephen
commented
May 30, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
doris-robot
commented
May 30, 2025
TPC-H: Total hot run time: 33799 ms |
doris-robot
commented
May 30, 2025
TPC-DS: Total hot run time: 194098 ms |
doris-robot
commented
May 30, 2025
ClickBench: Total hot run time: 29.86 s |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
We're closing this PR because it hasn't been updated in a while. |
What problem does this PR solve?
Related PR: #42004
orc submodule pr: apache/doris-thirdparty#313
Problem Summary:
In the previous PR #42004 , the optimization of orc tiny stripe was introduced. However, if there are many columns in the orc file and only a few of them are used in a query, the tiny stripe optimization will cause serious read amplification.
This PR introduces the
orc_tiny_stripe_amplification_factorsession variable to adjust the usage scenario of orc tiny stripe optimization. When the actual number of bytes to be read accounts for a larger proportion of the entire stripe than this parameter, read optimization is used. The default value of this parameter is 0.4, and the minimum value is 0.Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)