Feature/valkey document store

[daric93]

Related Issues

Valkey Document Store
#2574

Proposed Changes:

Implemented ValkeyDocumentStore as a built-in integration for Haystack. This component follows Haystack’s DocumentStore interface using Valkey’s search module for vector similarity and metadata-based filtering.

Valkey supports all essential capabilities required by the DocumentStore interface.
How did you test it?

• unit tests
• integration tests
• manual test (running example with haystack integration)

Checklist

• I have read the contributors guidelines and the code of conduct
• I have created the related issue
• I added tests and updated the docstrings

[CLAassistant]

All committers have signed the CLA.

[sjrl]

@daric93 thanks for the contribution!

Some feedback:

• Update the labeler.yml file to include integration:valkey
• Remove the file integrations/__init__.py which is here. This is purposely omitted since this messes up how namespacing works for our core integration packages.

I'll continue with a more in-depth review, but we are quite busy so this might take some time.

[sjrl]

Also the test folder structure should be updated. We follow the pattern that there should be one test file per real file in the integration. So we should only have:

• test_document_store.py
• test_document_store_async.py
• test_embedding_retriever.py
• test_filters.py

So rename your existing files and merge the other ones (test_static_methods.py and test_dict_converters.py)

[sjrl]

Thanks for the second round of review! I have addressed all the comments. Please take another look when you have time.

Thanks! I should have time to give it another look early next week.

[sjrl]

One last additional comment. Could you update this file https://github.com/deepset-ai/haystack-core-integrations/blob/main/README.md to include a row in the table for this new integratiom

[daric93]

One last additional comment. Could you update this file https://github.com/deepset-ai/haystack-core-integrations/blob/main/README.md to include a row in the table for this new integratiom

added row for valkey

[daric93]

Thanks for the feedback! I've addressed new comments.
Please review again when you have a chance.

[sjrl]

Thanks for the contribution!