From 54a26c60eaa2d717780e167939651397e7ff5a74 Mon Sep 17 00:00:00 2001 From: qidaye Date: Thu, 6 Feb 2025 19:06:27 +0800 Subject: [PATCH] opt --- be/src/olap/compaction.cpp | 2 +- be/src/olap/tablet_schema.cpp | 2 +- be/src/olap/tablet_schema.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/be/src/olap/compaction.cpp b/be/src/olap/compaction.cpp index 34e0e5e0027fdc..15ad98784dd8aa 100644 --- a/be/src/olap/compaction.cpp +++ b/be/src/olap/compaction.cpp @@ -837,7 +837,7 @@ void Compaction::construct_index_compaction_columns(RowsetWriterContext& ctx) { continue; } auto col_unique_id = col_unique_ids[0]; - if (!_cur_tablet_schema->exist_column_by_uid(col_unique_id)) { + if (!_cur_tablet_schema->has_column_unique_id(col_unique_id)) { LOG(WARNING) << "tablet[" << _tablet->tablet_id() << "] column_unique_id[" << col_unique_id << "] not found, will skip index compaction"; continue; diff --git a/be/src/olap/tablet_schema.cpp b/be/src/olap/tablet_schema.cpp index caf9a708b64991..82bc268d6a3497 100644 --- a/be/src/olap/tablet_schema.cpp +++ b/be/src/olap/tablet_schema.cpp @@ -1360,7 +1360,7 @@ bool TabletSchema::exist_column(const std::string& field_name) const { return _field_name_to_index.contains(StringRef {field_name}); } -bool TabletSchema::exist_column_by_uid(int32_t col_unique_id) const { +bool TabletSchema::has_column_unique_id(int32_t col_unique_id) const { return _field_id_to_index.contains(col_unique_id); } diff --git a/be/src/olap/tablet_schema.h b/be/src/olap/tablet_schema.h index cd2454985b6c12..fbb135188a03ef 100644 --- a/be/src/olap/tablet_schema.h +++ b/be/src/olap/tablet_schema.h @@ -345,7 +345,7 @@ class TabletSchema : public MetadataAdder { Result column(const std::string& field_name) const; Status have_column(const std::string& field_name) const; bool exist_column(const std::string& field_name) const; - bool exist_column_by_uid(int32_t col_unique_id) const; + bool has_column_unique_id(int32_t col_unique_id) const; const TabletColumn& column_by_uid(int32_t col_unique_id) const; TabletColumn& mutable_column_by_uid(int32_t col_unique_id); TabletColumn& mutable_column(size_t ordinal);