feat: adding delete_by_filter and update_by_filter to WeaviateDocumentStore

[davidsbatista]

Related Issues

• fixes add update_by_filter() and delete_by_filter() operations to WeaviateDocumentStore #2335

Proposed Changes:

• added delete_by_filter and update_by_filter methods to Weaviate document store
• Implemented sync and async versions of both methods

How did you test it?

• added tests for sync (test_document_store.py) and async (test_document_store_async.py) versions, following the same pattern as other document stores (OpenSearch, Elasticsearch)

Checklist

• I have read the contributors guidelines and the code of conduct
• I have updated the related issue with new insights and changes
• I added unit tests and updated the docstrings
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

[sjrl]

@davidsbatista looks good! Just a few minor comments.

One other thing that would be nice if straightforward to do is to test the pagination logic in your new methods. So to make sure that if we need to paginate everything still works as expected. Would this be straightforward to do?

[davidsbatista]

@sjrl added tests for the pagination as well - there's also other small fixes one can do, not directly related with this PR - I can do them after this one is merged.

[sjrl]

@sjrl added tests for the pagination as well - there's also other small fixes one can do, not directly related with this PR - I can do them after this one is merged.

Thanks and sounds good!

[sjrl]

Looks good!