Isolate Antfly ANN benchmark from default full-text work - #13
Conversation
|
We do sometimes need that full-text index [codex] Blocker for filtered runs: #13 deletes an index that older Antfly needs for filtersVectorDBBench #13 (#13) deletes For ordinary unfiltered ANN, that is appropriate. For filtered ANN, it is unsafe:
[INFERENCE] On older Antfly, deleting the only full-text index can make filtered ANN fail or Recommendation: delete the default index only for an explicitly unfiltered ANN case. At if not self.with_scalar_labels and not self._keep_default_full_text_index():
self._remove_default_full_text_index(client)A workload/capability flag would be cleaner than inferring solely from with_scalar_labels, Add an end-to-end filtered test against every supported Antfly generation before allowing |
|
[wrong PR] |
|
Looks like this PR addresses this concern and we can just set the flag. Merging. |
Summary
ANTFLY_VDBBENCH_KEEP_DEFAULT_FULL_TEXT=1sync_level=writedoes not wait for full-text indexing, but the full-text worker still consumes replay, storage, CPU, and memory. Removing it through the same public API used by a client isolates the vector benchmark; it does not change batch size or bypass Antfly's server.Validation
The repository venv does not currently include pytest, so the focused pytest file could not be invoked locally. Both full E2E runs exercised the removal path and received a successful public API response.
Main Antfly optimization: antflydb/antfly#512