vanna-ai / vanna

🤖 Chat with your SQL database 📊. Accurate Text-to-SQL Generation via LLMs using RAG 🔄.
https://vanna.ai/docs/
MIT License
9.36k stars 691 forks source link

Vanna with qdrant using old endpoints from qdrant v.1.0.0 or before #444

Closed mohamedamine456 closed 1 month ago

mohamedamine456 commented 1 month ago

Describe the bug I am using vanna.ai with a qdrant vector db running in a docker container, and when you try to verify if collection exists using "GET /collections/sql/exists" I get this error in the qdrant container logs "GET /collections/sql/exists HTTP/1.1" 404 which means, the endpoint does not exist, and I think bcs you are based on an old version of qdrant 1.0.0 or before, but the new version uses GET "/collections/{collection_name}" this

To Reproduce Steps to reproduce the behavior:

  1. use vanna.ai with newest version of qdrant database
  2. try the example code given in the docs to use vanna with qdrant database

Expected behavior to access qdrant database and find/create the needed collection

Error logs/Screenshots If applicable, add logs/screenshots to give more information about the issue.

Desktop (please complete the following information where):

Additional context also the endpoints used to create new collection changed

zainhoda commented 1 month ago

@Anush008 could you kindly confirm?

Anush008 commented 1 month ago

Hey. The collection exists API was introduced in a recent version.

https://api.qdrant.tech/api-reference/collections/collection-exists

Please update your Docker image.

mohamedamine456 commented 1 month ago

I was using the version 1.6.1 of qdrant, I ll update it to latest, thank you