deepset-ai / haystack-core-integrations

Additional packages (components, document stores and the likes) to extend the capabilities of Haystack version 2.0 and onwards
https://haystack.deepset.ai
Apache License 2.0
121 stars 119 forks source link

refactor!: Qdrant - remove `index` parameter from methods #1160

Closed anakin87 closed 3 weeks ago

anakin87 commented 3 weeks ago

Some methods of Qdrant Document Store allow specifying the index parameter.

This comes from the Haystack 1.x implementation. In 2.x, we are enforcing the convention where a Document Store corresponds to a specific index/collection.

Having some methods where you can specify index is inconsistent and fuels user confusion. See https://github.com/deepset-ai/haystack-core-integrations/issues/942

Proposed Changes:

How did you test it?

CI

Checklist