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

lide 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 102f27fa130 [fix](be) be core dump caused by an invalid pointer 
(#39430)
102f27fa130 is described below

commit 102f27fa130ff6da05142942bf1a841ee3c4bb45
Author: lw112 <131352377+felixw...@users.noreply.github.com>
AuthorDate: Thu Aug 15 19:35:09 2024 +0800

    [fix](be) be core dump caused by an invalid pointer (#39430)
---
 be/src/olap/rowset/segment_v2/segment.cpp | 2 ++
 be/src/olap/rowset/segment_v2/segment.h   | 5 -----
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/be/src/olap/rowset/segment_v2/segment.cpp 
b/be/src/olap/rowset/segment_v2/segment.cpp
index 90a8f9b447e..b56d607c855 100644
--- a/be/src/olap/rowset/segment_v2/segment.cpp
+++ b/be/src/olap/rowset/segment_v2/segment.cpp
@@ -229,6 +229,8 @@ Status Segment::load_index() {
 }
 
 Status Segment::_create_column_readers() {
+    std::unordered_map<uint32_t, uint32_t> _column_id_to_footer_ordinal;
+
     for (uint32_t ordinal = 0; ordinal < _footer.columns().size(); ++ordinal) {
         auto& column_pb = _footer.columns(ordinal);
         _column_id_to_footer_ordinal.emplace(column_pb.unique_id(), ordinal);
diff --git a/be/src/olap/rowset/segment_v2/segment.h 
b/be/src/olap/rowset/segment_v2/segment.h
index 461c3ba9451..00d3b906bb5 100644
--- a/be/src/olap/rowset/segment_v2/segment.h
+++ b/be/src/olap/rowset/segment_v2/segment.h
@@ -133,11 +133,6 @@ private:
     int64_t _meta_mem_usage;
     SegmentFooterPB _footer;
 
-    // Map from column unique id to column ordinal in footer's ColumnMetaPB
-    // If we can't find unique id from it, it means this segment is created
-    // with an old schema.
-    std::unordered_map<uint32_t, uint32_t> _column_id_to_footer_ordinal;
-
     // map column unique id ---> column reader
     // ColumnReader for each column in TabletSchema. If ColumnReader is 
nullptr,
     // This means that this segment has no data for that column, which may be 
added


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

Reply via email to