search

1Panel-dev / MaxKB

🚀 MaxKB 是一款基于大语言模型和 RAG 的开源知识库问答系统，广泛应用于智能客服、企业内部知识库、学术研究与教育等场景。
https://maxkb.cn/
GNU General Public License v3.0
11.59k stars 1.52k forks source link

[BUG]复杂编排获取下一个执行节点的逻辑是不是效率比较低 #1579

Open hello1534 opened 2 weeks ago

hello1534 commented 2 weeks ago

联系方式

No response

MaxKB 版本

1.7.0

问题描述

image 看代码中目前执行完当前节点，获取下个节点的逻辑，是需要去遍历所有的边，然后判断source_node_id是否是本节点，每次执行完一个节点后都需要去遍历一次，当应用比较复杂时感觉性能会较差。 为什么不在节点上加两个list字段，分别存储节点连接的左边和右边的集合，这样不是性能会更好吗。还是有其他点需要考虑，所以只能够使用遍历这种方法呢。

重现步骤

无

期待的正确结果

No response

相关日志输出

No response

附加信息

No response

shaohuzhang1 commented 2 weeks ago

好的，想贡献此增强功能吗？

hello1534 commented 2 weeks ago

好的，想贡献此增强功能吗？

嗯 不过不清楚改动会不会影响项目的其他功能，所以想先确认下之前这样设计，是不是有什么特殊的考虑点，还是只是因为实现起来比较简单，性能差点也能够接受就没有去优化了