Skip noindex fields in indexer (apache#2223)

Co-authored-by: hulk <[email protected]>
Beihao-Zhou · Apr 5, 2024 · 8ecccdd · 8ecccdd
1 parent d7b8450
commit 8ecccdd
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/search/indexer.cc b/src/search/indexer.cc
@@ -96,6 +96,10 @@ StatusOr<IndexUpdater::FieldValues> IndexUpdater::Record(std::string_view key, c
 
   FieldValues values;
   for (const auto &[field, info] : fields) {
+    if (info->noindex) {
+      continue;
+    }
+
     std::string value;
     auto s = retriever.Retrieve(field, &value);
     if (s.IsNotFound()) continue;
@@ -205,7 +209,11 @@ Status IndexUpdater::UpdateIndex(const std::string &field, std::string_view key,
 Status IndexUpdater::Update(const FieldValues &original, std::string_view key, const std::string &ns) {
   auto current = GET_OR_RET(Record(key, ns));
 
-  for (const auto &[field, _] : fields) {
+  for (const auto &[field, info] : fields) {
+    if (info->noindex) {
+      continue;
+    }
+
     std::string_view original_val, current_val;
 
     if (auto it = original.find(field); it != original.end()) {