Uh oh!
There was an error while loading. Please reload this page.
[support](orc)support orc file meta cache. - #54591
Conversation
Thearas
commented
Aug 11, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
hubgeter
commented
Aug 11, 2025
run buildall |
doris-robot
commented
Aug 11, 2025
TPC-H: Total hot run time: 33897 ms |
doris-robot
commented
Aug 11, 2025
TPC-DS: Total hot run time: 184447 ms |
doris-robot
commented
Aug 11, 2025
ClickBench: Total hot run time: 32.46 s |
| return buf.str(); | ||
| } | ||
| void FieldDescriptor::iceberg_sanitize(const std::vector<std::string>& read_columns) { |
There was a problem hiding this comment.
This code is currently unused.
reason :
Prior to pull request #27108, the Iceberg Parquet reader parsed the field ID and file name from the Parquet file's Properties iceberg.schema.
Since iceberg.schema stores the table name, which is not in valid Avro format, the PR #27108 needs to convert it (sanitize_avro_name).
In the current master code implementation, the field ID comes from the schema.

hubgeter
commented
Aug 12, 2025
run buildall |
doris-robot
commented
Aug 12, 2025
TPC-H: Total hot run time: 33906 ms |
doris-robot
commented
Aug 12, 2025
TPC-DS: Total hot run time: 184961 ms |
doris-robot
commented
Aug 12, 2025
ClickBench: Total hot run time: 32.91 s |
hubgeter
commented
Aug 13, 2025
run buildall |
doris-robot
commented
Aug 13, 2025
TPC-H: Total hot run time: 33751 ms |
doris-robot
commented
Aug 13, 2025
TPC-DS: Total hot run time: 184313 ms |
doris-robot
commented
Aug 13, 2025
ClickBench: Total hot run time: 32.26 s |
doris-robot
commented
Aug 13, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 13, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
| : _file_reader; | ||
| } | ||
| if (_file_metadata) { | ||
| std::cout << "_file_metadata not null\n"; |
cbc1456 to
b6aea5bComparehubgeter
commented
Aug 19, 2025
run buildall |
doris-robot
commented
Aug 19, 2025
TPC-H: Total hot run time: 33785 ms |
doris-robot
commented
Aug 19, 2025
TPC-DS: Total hot run time: 184179 ms |
doris-robot
commented
Aug 19, 2025
ClickBench: Total hot run time: 32.7 s |
hubgeter
commented
Aug 20, 2025
run buildall |
doris-robot
commented
Aug 20, 2025
TPC-H: Total hot run time: 34002 ms |
doris-robot
commented
Aug 20, 2025
TPC-DS: Total hot run time: 185159 ms |
doris-robot
commented
Aug 20, 2025
ClickBench: Total hot run time: 32.41 s |
doris-robot
commented
Aug 20, 2025
BE 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. |
morningman
commented
Aug 28, 2025
run check_coverage |
Uh oh!
There was an error while loading. Please reload this page.
Problem Summary: This PR includes three changes: 1. Support for file meta cache for ORC files. 2. Changed the file meta cache key from `file name + modification time` to `file name + modification time / file size` , reduce the chance of reading old meta. 3. Removed some unused code in the parquet meta. 4. Users can use profile to observe whether the cache hits or not. `FileFooterHitCache`: hit cache. `FileFooterReadCalls`: not hit cache or disable cache. BTW : disable cache : be conf `max_external_file_meta_cache_num` <= 0
What problem does this PR solve?
Problem Summary:
This PR includes three changes:
file name + modification timetofile name + modification time / file size, reduce the chance of reading old meta.FileFooterHitCache: hit cache.FileFooterReadCalls: not hit cache or disable cache.BTW : disable cache : be conf
max_external_file_meta_cache_num<= 0Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)