[Fix](segment iterator) use correct column_id (apache#36313)

previous PR apache#36277 use wrong column_id to access `_storage_name_and_type` and may result nullptr
eldenmoon · Jun 19, 2024 · f3d06da · f3d06da
1 parent 5b0620d
commit f3d06da
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/be/src/olap/rowset/segment_v2/segment_iterator.cpp b/be/src/olap/rowset/segment_v2/segment_iterator.cpp
@@ -980,9 +980,9 @@ std::string SegmentIterator::_gen_predicate_result_sign(ColumnPredicateInfo* pre
     if (column_desc->path() != nullptr) {
         const static std::string pattern = "(CAST {}(Nullable(Variant)) TO {})";
         // indicate a subcolumn access for variant, using the expression pattern as pred result sign name
-        col_name =
-                fmt::format(pattern, col_name,
-                            _storage_name_and_type[predicate_info->column_id].second->get_name());
+        col_name = fmt::format(pattern, col_name,
+                               _storage_name_and_type[_schema->column_id(predicate_info->column_id)]
+                                       .second->get_name());
     }
     std::string pred_result_sign;
     pred_result_sign = BeConsts::BLOCK_TEMP_COLUMN_PREFIX + col_name + "_" +