semantic-kernel icon indicating copy to clipboard operation
semantic-kernel copied to clipboard

Qdrant memory store connector should also support using the Qdrant cloud (taking the API Key to initialize the connection)

Open anirudhgarg opened this issue 1 year ago • 4 comments

Currently the Qdrant memory store connector only supports local installation and thus asking for the port and the endpoint. It should also support the Qdrant cloud service taking the API Key and the endpoint - see here: https://pypi.org/project/qdrant-client/#:~:text=To%20connect%20to%20Qdrant%20server%2C%20simply%20specify%20host,with%20docker%3A%20docker%20run%20-p%206333%20%3A6333%20qdrant%2Fqdrant%3Alatest

anirudhgarg avatar May 01 '23 04:05 anirudhgarg

Thanks anirudhgarg, we'll look into adding this into the backlog, but of course we are open to contributions from the community as well.

microsoftShannon avatar May 02 '23 20:05 microsoftShannon

Thanks for considering it. The reason for supporting this is then this allows folks to deploy their application using Semantic Kernel which inturn uses Qdrant cloud - as a service.

anirudhgarg avatar May 03 '23 21:05 anirudhgarg

Can't this be achieved in the current design by passing an instance of HttpClient with your own handler in:

var httpClient = new HttpClient();
// add request-modification logic here to add the api key to request headers on this HttpClient instance.

var qdrantMemoryStore = new QdrantMemoryStore(
    new QdrantVectorDbClient(endpoint, vectorSize, httpClient: httpClient));

x3haloed avatar May 10 '23 20:05 x3haloed

@tawalke can you take a look at this issue

evchaki avatar May 16 '23 21:05 evchaki

@adrianwyatt This is supported now, isn't it?

glahaye avatar Jul 27 '23 05:07 glahaye

Looks qdrant memory store just use the qdrant python sdk, but need more parameters in init function of QdrantMemoryStore.

So, I just set the _qdrantclient field with a qdrant_client object to set the api-key for now.

qc

breakstring avatar Sep 10 '23 07:09 breakstring

This issue is very stale. If this item is still needed, please feel free to re-open the issue with the appropriate comment; otherwise, please file a new issue.

moonbox3 avatar Mar 08 '24 19:03 moonbox3