Prhub

#48014 [Core] Move MRV1 `late_interaction_runner.py` out of MRV2 subtree

原始 PR 作者 njhill 合并时间 2026-07-09 02:30 文件变更 3 提交数 1 评论 1 代码增减 +2 / -2

执行摘要

移动 late_interaction_runner.py 到正确路径

该文件在 PR #36159 中被添加到 vllm/v1/worker/gpu/pool/,但这是 MRV2 的源码子树,而该文件属于 MRV1 核心模块。将其移出可避免混淆,保持代码结构清晰。

建议快速合并。这是一个纯代码结构清理 PR,无功能影响,但有助于减少未来开发者的困惑。

讨论亮点

无实质性讨论,只有一个自动化 comment 和 WoosukKwon 的批准。

实现拆解

  1. 移动文件:将 vllm/v1/worker/gpu/pool/late_interaction_runner.py 重命名为 vllm/v1/pool/late_interaction_runner.py,文件内容不变。
  2. 更新 gpu_model_runner.py 中的导入:将 from vllm.v1.worker.gpu.pool.late_interaction_runner import LateInteractionRunner 替换为 from vllm.v1.pool.late_interaction_runner import LateInteractionRunner,并保持导入顺序。
  3. 更新测试文件中的导入:将 tests/v1/worker/test_late_interaction_runner.py 中的导入路径同步更新。
  4. 无逻辑变更:只涉及文件路径和导入语句的调整,不改变任何运行时行为。
文件 模块 状态 重要度
vllm/v1/pool/late_interaction_runner.py 核心库 renamed 4.58
vllm/v1/worker/gpu_model_runner.py 核心库 modified 5.17
tests/v1/worker/test_late_interaction_runner.py 测试 modified 3.7

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

评论区精华

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

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

风险与影响

风险极低。仅涉及文件重命名和导入路径更新,代码逻辑和执行路径完全不变。但需要确保 CI 中的测试能够正确发现新的测试文件位置(但测试文件本身未移动,只是导入变更)。

影响范围极小:仅影响直接导入旧路径的模块(gpu_model_runner.py 和测试文件)。对用户无感知,对系统行为无影响,有助于保持代码组织的一致性。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论