From 6eb746d994f6f5a7f9d259989df3b94b6bbdf87a Mon Sep 17 00:00:00 2001 From: Weny Xu Date: Mon, 13 Jan 2025 13:15:20 +0800 Subject: [PATCH] fix: skip building indexer when `indexed_column_ids` are empty (#5348) --- src/mito2/src/sst/index.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/mito2/src/sst/index.rs b/src/mito2/src/sst/index.rs index dc0f0978f8..4d45e21a62 100644 --- a/src/mito2/src/sst/index.rs +++ b/src/mito2/src/sst/index.rs @@ -222,9 +222,12 @@ impl<'a> IndexerBuilder<'a> { return None; } - if self.metadata.primary_key.is_empty() { + let indexed_column_ids = self.metadata.inverted_indexed_column_ids( + self.index_options.inverted_index.ignore_column_ids.iter(), + ); + if indexed_column_ids.is_empty() { debug!( - "No tag columns, skip creating index, region_id: {}, file_id: {}", + "No columns to be indexed, skip creating inverted index, region_id: {}, file_id: {}", self.metadata.region_id, self.file_id, ); return None; @@ -259,9 +262,7 @@ impl<'a> IndexerBuilder<'a> { self.intermediate_manager.clone(), self.inverted_index_config.mem_threshold_on_create(), segment_row_count, - self.metadata.inverted_indexed_column_ids( - self.index_options.inverted_index.ignore_column_ids.iter(), - ), + indexed_column_ids, ); Some(indexer)