This is an automated email from the ASF dual-hosted git repository.
eldenmoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 2040505d37e [Chore](Segment) avoid print stack trace when
column_unique_id not found (#56021)
2040505d37e is described below
commit 2040505d37e1114449cc8785ca44be9fc230409e
Author: lihangyu <[email protected]>
AuthorDate: Sat Sep 13 14:37:06 2025 +0800
[Chore](Segment) avoid print stack trace when column_unique_id not found
(#56021)
---
be/src/olap/rowset/segment_v2/column_reader_cache.cpp | 6 ++++--
be/src/olap/rowset/segment_v2/segment.cpp | 6 ++++--
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/be/src/olap/rowset/segment_v2/column_reader_cache.cpp
b/be/src/olap/rowset/segment_v2/column_reader_cache.cpp
index 3ebf8b86e25..62719459ecf 100644
--- a/be/src/olap/rowset/segment_v2/column_reader_cache.cpp
+++ b/be/src/olap/rowset/segment_v2/column_reader_cache.cpp
@@ -113,7 +113,8 @@ Status ColumnReaderCache::get_column_reader(int32_t col_uid,
if (it == _segment->_column_uid_to_footer_ordinal.end()) {
// no such column in this segment, return nullptr
*column_reader = nullptr;
- return Status::NotFound("column not found in segment, col_uid={}",
col_uid);
+ return Status::Error<ErrorCode::NOT_FOUND, false>("column not found in
segment, col_uid={}",
+ col_uid);
}
std::shared_ptr<SegmentFooterPB> footer_pb_shared;
{
@@ -180,7 +181,8 @@ Status ColumnReaderCache::get_path_column_reader(uint32_t
col_uid,
}
// no such column in this segment, return nullptr
*column_reader = nullptr;
- return Status::NotFound("column not found in segment, col_uid={}",
col_uid);
+ return Status::Error<ErrorCode::NOT_FOUND, false>("column not found in
segment, col_uid={}",
+ col_uid);
}
#include "common/compile_check_end.h"
diff --git a/be/src/olap/rowset/segment_v2/segment.cpp
b/be/src/olap/rowset/segment_v2/segment.cpp
index f006320c1c4..168484dea37 100644
--- a/be/src/olap/rowset/segment_v2/segment.cpp
+++ b/be/src/olap/rowset/segment_v2/segment.cpp
@@ -757,7 +757,8 @@ Status Segment::get_column_reader(int32_t col_uid,
std::shared_ptr<ColumnReader>
// The column is not in this segment, return nullptr
if (!_tablet_schema->has_column_unique_id(col_uid)) {
*column_reader = nullptr;
- return Status::NotFound("column not found in segment, col_uid={}",
col_uid);
+ return Status::Error<ErrorCode::NOT_FOUND, false>("column not found in
segment, col_uid={}",
+ col_uid);
}
return _column_reader_cache->get_column_reader(col_uid, column_reader,
stats);
}
@@ -771,7 +772,8 @@ Status Segment::get_column_reader(const TabletColumn& col,
// The column is not in this segment, return nullptr
if (!_tablet_schema->has_column_unique_id(col_uid)) {
*column_reader = nullptr;
- return Status::NotFound("column not found in segment, col_uid={}",
col_uid);
+ return Status::Error<ErrorCode::NOT_FOUND, false>("column not found in
segment, col_uid={}",
+ col_uid);
}
if (col.has_path_info()) {
vectorized::PathInData relative_path =
col.path_info_ptr()->copy_pop_front();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
