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

Support qdrant vector database #1692

Closed
wants to merge 6 commits into from

Conversation

Buliqioqiolibusdo
Copy link

支持qdrant向量库 (#1304 )

Copy link

cla-assistant bot commented Jun 4, 2024

CLA assistant check
All committers have signed the CLA.

@c121914yu
Copy link
Collaborator

🤖 Generated by deploy action

👀 Visit Preview

@c121914yu
Copy link
Collaborator

c121914yu commented Jun 5, 2024

混有无效的内容~
这个pr无法合并~ milvus已经是 top1 向量库了,再支持其他的就增加维护压力了。。 写个接入其他向量库文档差不多了

@Buliqioqiolibusdo
Copy link
Author

主程你好,milvus和pg都有各自的client依赖库参与项目编译,如何通过文档来指引注入qdrant的依赖项,请指教~
另外getVectorDataByTime函数占位是由于项目急于上测试, 如果补全能否合并pr

@c121914yu
Copy link
Collaborator

Author

可以通过教学教程方式提交~
合入向量库,在未来修改字段或索引等操作时,会增加负担,由于字段变更和索引变更不像crud操作可以适配起来,我们需要为每个数据库都去写变更操作并测试。。qdrant我们没人用过,可能会导致未来无法更新。

@Buliqioqiolibusdo
Copy link
Author

教学文档目测不太能描述的清楚,涉及好几个文件,并且更改完需要开发人员自行编译打包镜像,能否考虑分支或者tag的方式保留这个版本,后续的更新我会持续迭代测试qdrant相关的代码.
另外租户功能在企业内合作方面很重要,而我们短期内不方便切dify使用,能否贡献这部分相关的代码到开源社区,或者是否允许我基于当前版本另起一个项目?

@c121914yu
Copy link
Collaborator

教学文档目测不太能描述的清楚,涉及好几个文件,并且更改完需要开发人员自行编译打包镜像,能否考虑分支或者tag的方式保留这个版本,后续的更新我会持续迭代测试qdrant相关的代码. 另外租户功能在企业内合作方面很重要,而我们短期内不方便切dify使用,能否贡献这部分相关的代码到开源社区,或者是否允许我基于当前版本另起一个项目?

租户在开源版不可能开放的~ qdrant这个,我不确定你为什么要选择它,以及加入他的意义。

@c121914yu c121914yu closed this Jun 19, 2024
@c121914yu
Copy link
Collaborator

确定不合了,后续还计划在向量库进行更多操作,维护起来麻烦,甚至考虑只支持 milvus 即可,最近 milvus 发布了 milvus lite更轻量,也在和他们团队在沟通。
milvus团队都是国内的,沟通起来也方便,给了我们不少技术支持。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants