Prhub

#28389 [UnifiedTree]: Clean up some unused dead code.

原始 PR 作者 hzh0425 合并时间 2026-06-16 21:53 文件变更 1 提交数 1 评论 10 代码增减 +0 / -28

执行摘要

清除 UnifiedTree 中三个未使用的死代码符号

PR body 未明确说明动机,但从变更内容看是为了清理 UnifiedRadixCache 中不再使用的死代码,减轻维护负担。作者通过 CI 触发相关测试组(radix_cache/unified_radix_tree)并全部通过确认无害。

建议快速合并。此类死代码清理有助于保持核心模块的可维护性,且风险经过 CI 验证。

讨论亮点

无 review 评论。作者在评论中多次触发 CI 重跑相关测试组,最终所有测试通过后自行合并。

实现拆解

  1. 删除 all_swa_values_flatten 方法:原方法约 4 行,内部委托 _all_component_values_flatten(ComponentType.SWA),不再被任何位置引用。
  2. 删除 _rebuild_host_leaf_sets 方法:约 7 行,用于 L1-only reset 后重建 evictable_host_leaves,但调用点已被移除或从未被实际使用。
  3. 删除 _rebuild_host_lru_lists 方法:约 17 行,用于重建额外组件的 host LRU 列表,同样已无调用。
  4. CI 验证:通过 /rerun-group/rerun-test 命令多次触发 radix_cache/unified_radix_tree 下的 3 个测试以及单元测试和基准测试,结果全部通过。
文件 模块 状态 重要度
python/sglang/srt/mem_cache/unified_radix_cache.py 缓存层 modified 6.2

关键符号

all_swa_values_flatten _rebuild_host_leaf_sets _rebuild_host_lru_lists

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低。被删除的三个函数均无任何引用点(IDE 搜索或 grep 无法找到调用),CI 测试全部通过。但若未来有规划引入它们的功能,需重新实现。

仅影响 python/sglang/srt/mem_cache/unified_radix_cache.py 文件,减少约 28 行死代码。对外部用户和系统行为无任何影响。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论