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: add support for qdrant foreign data wrapper #156

Closed
wants to merge 21 commits into from
Closed

Conversation

imor
Copy link
Contributor

@imor imor commented Sep 20, 2023

This PR adds a foreign data wrapper for qdrant vector database. This is a bare-bones version only supporting simple select statements which returns data from the Qdrant's scroll points endpoint. For more information about how the wrapper works, read the docs in the PR.

Implementation wise, this wrapper is quite similar to others in design in that we call the collection/<collection_name>/points/scroll api with appropriate parameters in the wrapper callbacks and return that data in a form Postgres is happy with. One slight improvement is that we run the query in batches so as to not balloon the memory used by the wrapper. The default batch size is currently hardcoded to 1000.

@imor imor changed the title Feat/qdrant fdw add support for qdrant foreign data wrapper Sep 21, 2023
@imor imor marked this pull request as ready for review September 26, 2023 13:07
@imor imor requested review from burmecia and olirice and removed request for burmecia September 26, 2023 13:08
Copy link
Collaborator

@olirice olirice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

*On Hold until Oct 14

@imor imor changed the title add support for qdrant foreign data wrapper feat: add support for qdrant foreign data wrapper Oct 4, 2023
@olirice olirice closed this Oct 13, 2023
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.

2 participants