Skip to content

Commit c05db7d

Browse files
authored
[fix](parquet-reader) Fix read parquet footer is missing in FileReadBytes profile. (#59571)
### What problem does this PR solve? Issue Number: close #xxx Related PR: #xxx Problem Summary: ### Release note Cherry-pick #58383 ### Check List (For Author) - Test <!-- At least one of them must be included. --> - [ ] Regression test - [ ] Unit Test - [ ] Manual test (add detailed scripts or steps below) - [ ] No need to test or manual test. Explain why: - [ ] This is a refactor/code format and no logic has been changed. - [ ] Previous test can cover this change. - [ ] No code files have been changed. - [ ] Other reason <!-- Add your reason? --> - Behavior changed: - [ ] No. - [ ] Yes. <!-- Explain the behavior change --> - Does this need documentation? - [ ] No. - [ ] Yes. <!-- Add document PR link here. eg: apache/doris-website#1214 --> ### Check List (For Reviewer who merge this PR) - [ ] Confirm the release note - [ ] Confirm test cases - [ ] Confirm document - [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
1 parent 3fc1c03 commit c05db7d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

be/src/vec/exec/format/parquet/vparquet_reader.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,8 +258,8 @@ Status ParquetReader::_open_file() {
258258
const auto& file_meta_cache_key =
259259
FileMetaCache::get_key(_tracing_file_reader, _file_description);
260260
if (!_meta_cache->lookup(file_meta_cache_key, &_meta_cache_handle)) {
261-
RETURN_IF_ERROR(parse_thrift_footer(_file_reader, &_file_metadata_ptr, &meta_size,
262-
_io_ctx, enable_mapping_varbinary));
261+
RETURN_IF_ERROR(parse_thrift_footer(_tracing_file_reader, &_file_metadata_ptr,
262+
&meta_size, _io_ctx, enable_mapping_varbinary));
263263
// _file_metadata_ptr.release() : move control of _file_metadata to _meta_cache_handle
264264
_meta_cache->insert(file_meta_cache_key, _file_metadata_ptr.release(),
265265
&_meta_cache_handle);

0 commit comments

Comments
 (0)