已开启
[NL2SQL] MetaVisorClient 类与 schema_linking 方法死代码(90+ 行未使用) #32
henry创建于  7月7日
henry成员
7月7日 创建

现象

dataagent/agents/nl2sql/utils/metavisor_client.py:26-97MetaVisorClient整体是死代码

  • 生产路径只用了 ValueMatchClient(validator.py 调用)
  • MetaVisorClient 仅被 legacy actions/perceptor/perceptor_atomic.py 引用(已废弃)
  • MetaVisorClient.semantic_search_column(单数)和 semantic_search_tables 方法从未被调用

影响

  • 维护成本:90+ 行死代码,新人误以为还在用
  • 误导:类名 MetaVisorClient 暗示是 metavisor 调用的主入口,实际是 legacy

修复方案

任选其一:

  1. 删除 MetaVisorClient 类与两个死方法
  2. 迁移到 actions/perceptor/perceptor_atomic.py 内部(与唯一调用方同模块)

涉及文件

  • dataagent/agents/nl2sql/utils/metavisor_client.py:26-97

验证

  • rg "MetaVisorClient" dataagent/ tests/ 应只剩 ValueMatchClient 与测试引用,无 MetaVisorClient 残留。

严重性

major(90+ 行死代码)

环境

  • 分支:0623
  • HEAD:c5c899a4196717d1eb5944a9736f766310655ef7
likedislike
xsmqxsmq成员
7月20日 关联了看板:@hwxsmq的看板 20260720