[doris] branch branch-1.2-lts updated: [Fix](Parquet Reader)Branch 1.2 lts bugfix (#24094)

[morningman]

This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new 01d4aa726bd [Fix](Parquet Reader)Branch 1.2 lts bugfix (#24094)
01d4aa726bd is described below

commit 01d4aa726bd188a0980075fc91a728747cd2b238
Author: GoGoWen <82132356+gogo...@users.noreply.github.com>
AuthorDate: Sat Oct 7 13:53:34 2023 +0800

    [Fix](Parquet Reader)Branch 1.2 lts bugfix (#24094)
---
 be/src/exec/arrow/parquet_row_group_reader.cpp       | 2 +-
 be/src/vec/exec/format/parquet/parquet_thrift_util.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/be/src/exec/arrow/parquet_row_group_reader.cpp 
b/be/src/exec/arrow/parquet_row_group_reader.cpp
index a2f83c8f214..83b947ba621 100644
--- a/be/src/exec/arrow/parquet_row_group_reader.cpp
+++ b/be/src/exec/arrow/parquet_row_group_reader.cpp
@@ -173,7 +173,7 @@ Status RowGroupReader::init_filter_groups(const 
TupleDescriptor* tuple_desc,
                 // Column not exist in parquet file
                 continue;
             }
-            auto slot_iter = _slot_conjuncts.find(slot_id);
+            auto slot_iter = 
_slot_conjuncts.find(tuple_desc->slots()[slot_id]->id());
             if (slot_iter == _slot_conjuncts.end()) {
                 continue;
             }
diff --git a/be/src/vec/exec/format/parquet/parquet_thrift_util.h 
b/be/src/vec/exec/format/parquet/parquet_thrift_util.h
index b8be080d945..5642bbff040 100644
--- a/be/src/vec/exec/format/parquet/parquet_thrift_util.h
+++ b/be/src/vec/exec/format/parquet/parquet_thrift_util.h
@@ -50,7 +50,7 @@ static Status parse_thrift_footer(FileReader* file, 
std::shared_ptr<FileMetaData
     // get metadata_size
     uint32_t metadata_size = decode_fixed32_le(footer);
     if (metadata_size > file_size - PARQUET_FOOTER_SIZE) {
-        Status::Corruption("Parquet file size is ", file_size,
+        return Status::Corruption("Parquet file size is ", file_size,
                            " bytes, smaller than the size reported by footer's 
(", metadata_size,
                            "bytes)");
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org