Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Qdrant ANN backend #703

Closed
wants to merge 4 commits into from
Closed

feat: Qdrant ANN backend #703

wants to merge 4 commits into from

Conversation

Anush008
Copy link

@Anush008 Anush008 commented May 1, 2024

Description

This PR adds support for Qdrant to be used as an ANN backend. Currently, the integration is available as a separate package at https://github.com/qdrant/qdrant-txtai.

With the recent addition of PGVector, we hope txtai is now open to provide native support for external vector database providers.

Testing

Tests have been configured using an in-memory and Qdrant server instance.

@davidmezzetti
Copy link
Member

Thank you for this PR!

A recent objective of txtai was to add top level support for persisting data to relational databases. This is something I'm willing to maintain as a core component.

A lesson learned by langchain and llama_index is that putting everything into a single project gets messy fast.

I've noticed that the Qdrant txtai plugin hasn't been maintained. Is that component considered deprecated? If so, I'm not opposed to creating a plugin project for this.

@Anush008
Copy link
Author

Anush008 commented May 2, 2024

Hi David.
Thanks for taking a look. We saw the PGVector addition and the Qdrant integration needed some updates since a while, so thought of raising this PR.

If so, I'm not opposed to creating a plugin project for this.

Could you explain how that would work?

@davidmezzetti
Copy link
Member

I'd basically do what the original qdrant-txtai project does now. It would be it's own project.

I don't have the resources to maintain this as a core piece of the project. It would have to be something that I fix when I get to it, like what qdrant-txtai is now.

@Anush008
Copy link
Author

Anush008 commented May 6, 2024

We can contribute to the maintenance efforts if that works for you.

@davidmezzetti
Copy link
Member

Thanks. I'll keep this open until I figure out the best way to integrate this. I'd also like similar plugins for other vector databases and systems like Elasticsearch. It will get unwieldy if those are all in the core package.

@davidmezzetti
Copy link
Member

davidmezzetti commented May 21, 2024

It looks like these changes have been pushed into the qdrant-txtai project? If so, I'll close this as that is probably the best place for this to live for now.

@Anush008 Anush008 closed this May 21, 2024
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.

None yet

2 participants