Uh oh!
There was an error while loading. Please reload this page.
[External](parquet) pass non predicates column's offset index to RowGroupReader - #55795
Conversation
hello-stephen
commented
Sep 9, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
liutang123
commented
Sep 9, 2025
run buildall |
doris-robot
commented
Sep 9, 2025
TPC-H: Total hot run time: 34644 ms |
doris-robot
commented
Sep 9, 2025
TPC-DS: Total hot run time: 189549 ms |
doris-robot
commented
Sep 9, 2025
ClickBench: Total hot run time: 29.71 s |
f9b4486 to
960e2caCompareliutang123
commented
Sep 9, 2025
run buildall |
doris-robot
commented
Sep 9, 2025
TPC-H: Total hot run time: 34512 ms |
doris-robot
commented
Sep 9, 2025
TPC-DS: Total hot run time: 188475 ms |
960e2ca to
2d47b96Compareliutang123
commented
Sep 9, 2025
run buildall |
doris-robot
commented
Sep 9, 2025
ClickBench: Total hot run time: 30.54 s |
doris-robot
commented
Sep 9, 2025
TPC-H: Total hot run time: 34669 ms |
doris-robot
commented
Sep 9, 2025
TPC-DS: Total hot run time: 188775 ms |
doris-robot
commented
Sep 9, 2025
ClickBench: Total hot run time: 30.38 s |
2d47b96 to
2e39ffdCompareliutang123
commented
Sep 10, 2025
run buildall |
doris-robot
commented
Sep 10, 2025
TPC-H: Total hot run time: 34756 ms |
doris-robot
commented
Sep 10, 2025
TPC-DS: Total hot run time: 189533 ms |
doris-robot
commented
Sep 10, 2025
ClickBench: Total hot run time: 30.5 s |
doris-robot
commented
Sep 10, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Sep 10, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
2e39ffd to
c5f6b09Comparemorningman
commented
Oct 21, 2025
run buildall |
doris-robot
commented
Oct 21, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Oct 21, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
Uh oh!
There was an error while loading. Please reload this page.
…roupReader (#55795) ### What problem does this PR solve? Now, Parquet reader don't pass non predicate coloumn's offset index to `RowGroupReader`. `create_page_reader` will create `PageReader` rather than `PageReaderWithOffsetIndex`. The `PageReader` will parse each page header when `skip_page` and `next_page_header`. In the case of merging io, read and enlarge severely. So, in this PR, we pass the offset index to `RowGroupReader`. Co-authored-by: liutang123 <liulijia@gmail.com>
…roupReader (apache#55795) ### What problem does this PR solve? Now, Parquet reader don't pass non predicate coloumn's offset index to `RowGroupReader`. `create_page_reader` will create `PageReader` rather than `PageReaderWithOffsetIndex`. The `PageReader` will parse each page header when `skip_page` and `next_page_header`. In the case of merging io, read and enlarge severely. So, in this PR, we pass the offset index to `RowGroupReader`. Co-authored-by: liutang123 <liulijia@gmail.com>
…dex to RowGroupReader (apache#55795)
What problem does this PR solve?
Now, Parquet reader don't pass non predicate coloumn's offset index to
RowGroupReader.create_page_readerwill createPageReaderrather thanPageReaderWithOffsetIndex.The
PageReaderwill parse each page header whenskip_pageandnext_page_header. In the case of merging io, read and enlarge severely.So, in this PR, we pass the offset index to
RowGroupReader.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)