fix: Skip compaction when encouters permission error

Signed-off-by: yangxuan <xuan.yang@zilliz.com>

Author: XuanYang-cn

vectordb_bench/backend/clients/milvus/milvus.py

@@ -165,23 +165,27 @@ def _optimize(self):
         log.info(f"{self.name} optimizing before search")
         try:
             self.client.flush(self.collection_name)
-            self._wait_for_segments_sorted()
-            self._wait_for_index()
+
             if self.case_config.is_gpu_index:
                 log.debug("skip force merge compaction for gpu index type.")
             else:
                 try:
+                    # wait for sort, index, comapct
+                    self._wait_for_segments_sorted()
+                    self._wait_for_index()
                     compaction_id = self.client.compact(self.collection_name, target_size=(2**63 - 1))
                     if compaction_id > 0:
                         self._wait_for_compaction(compaction_id)
                     log.info(f"{self.name} force merge compaction completed.")
-                    self._wait_for_index()
                 except Exception as e:
-                    log.warning(f"{self.name} compact error: {e}")
+                    log.warning(f"{self.name} compact or list segments error: {e}")
                     if hasattr(e, "code") and e.code().name == "PERMISSION_DENIED":
-                        log.warning("Skip compact due to permission denied.")
+                        log.warning("Skip compact due to list segments or compact permission denied.")
                     else:
                         raise e from None
+
+            # wait for index no matter what
+            self._wait_for_index()
             self.client.refresh_load(self.collection_name)
         except Exception as e:
             log.warning(f"{self.name} optimize error: {e}")