feat: only use optimize reindex when the import is also true

marcosgz · Jul 19, 2024 · 4d4e9b8 · 4d4e9b8
1 parent 6787a6e
commit 4d4e9b8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/esse/index/indices.rb b/lib/esse/index/indices.rb
@@ -54,7 +54,7 @@ def reset_index(suffix: index_suffix, optimize: true, import: true, reindex: fal
         suffix ||= Esse.timestamp
         suffix = Esse.timestamp while index_exist?(suffix: suffix)
 
-        if optimize
+        if optimize && import
           definition = [settings_hash, mappings_hash].reduce(&:merge)
           number_of_replicas = definition.dig(Esse::SETTING_ROOT_KEY, :index, :number_of_replicas)
           refresh_interval = definition.dig(Esse::SETTING_ROOT_KEY, :index, :refresh_interval)
@@ -74,7 +74,7 @@ def reset_index(suffix: index_suffix, optimize: true, import: true, reindex: fal
           # @TODO: Reindex using the reindex API
         end
 
-        if optimize && number_of_replicas != new_number_of_replicas || refresh_interval != new_refresh_interval
+        if optimize && import && number_of_replicas != new_number_of_replicas || refresh_interval != new_refresh_interval
           update_settings(suffix: suffix)
           refresh(suffix: suffix)
         end