Skip to content

Conversation

@sobychacko
Copy link
Contributor

Add string-based filter deletion alongside the Filter.Expression-based deletion for Milvus vector store, providing consistent deletion capabilities with other vector store implementations.

Key changes:

  • Add delete(Filter.Expression) implementation for Milvus store
  • Leverage existing MilvusFilterExpressionConverter for filter translation
  • Use Milvus client's native delete API with filter expressions
  • Add comprehensive integration tests for filter deletion
  • Support both simple and complex filter expressions

This maintains consistency with other vector store implementations while utilizing Milvus-specific APIs for efficient metadata-based deletion.

Add string-based filter deletion alongside the Filter.Expression-based deletion
for Milvus vector store, providing consistent deletion capabilities with
other vector store implementations.

Key changes:
- Add delete(Filter.Expression) implementation for Milvus store
- Leverage existing MilvusFilterExpressionConverter for filter translation
- Use Milvus client's native delete API with filter expressions
- Add comprehensive integration tests for filter deletion
- Support both simple and complex filter expressions

This maintains consistency with other vector store implementations while
utilizing Milvus-specific APIs for efficient metadata-based deletion.
@sobychacko sobychacko added this to the 1.0.0-M6 milestone Jan 27, 2025
@markpollack markpollack merged commit 946657f into spring-projects:main Jan 28, 2025
2 checks passed
@markpollack markpollack self-assigned this Jan 28, 2025
@petros94
Copy link

Nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants