执行摘要

• 一句话：去除冗余的 req_pool_idx 赋值
• 推荐动作：建议精读以理解分配器与调度器之间的职责边界。该 PR 展示了如何通过消除重复来使数据流更清晰。

功能与动机

PR body 指出 alloc_for_extend → alloc_req_slots → ReqToTokenPool.alloc 已在每个请求上写入了 r.req_pool_idx（见 ReqToTokenPool.alloc 第 182-183 行 及 HybridReqToTokenPool 通过 super() 的委托）。循环顶部的 req.req_pool_idx = req_pool_indices[i] 是多余的重复赋值，应删除以让该字段仅由分配器内部设置。

实现拆解

1. 在 python/sglang/srt/managers/schedule_batch.py 的 prepare_for_extend 方法中，将 alloc_for_extend 的返回解包从 out_cache_loc, req_pool_indices_tensor, req_pool_indices 改为 out_cache_loc, req_pool_indices_tensor, _，表示 req_pool_indices 列表不再使用。
2. 同时删除循环内的赋值语句 req.req_pool_idx = req_pool_indices[i]。
3. 没有其他文件或配置变更。

关键文件：

• python/sglang/srt/managers/schedule_batch.py（模块 调度器；类别 source；类型 core-logic；符号 prepare_for_extend）: 唯一变更文件，修改了 prepare_for_extend 方法，移除了冗余的 req_pool_idx 赋值及对应的变量绑定。

关键符号：prepare_for_extend

关键源码片段

python/sglang/srt/managers/schedule_batch.py

唯一变更文件，修改了 prepare_for_extend 方法，移除了冗余的 req_pool_idx 赋值及对应的变量绑定。

# 修改前的代码段
out_cache_loc, req_pool_indices_tensor, req_pool_indices = alloc_for_extend(
    self
)
# ... 循环内部
req.req_pool_idx = req_pool_indices[i] # 此赋值已由 alloc_for_extend 内部完成
​
# 修改后的代码段：使用 _ 丢弃不需要的列表，删除冗余赋值
out_cache_loc, req_pool_indices_tensor, _ = alloc_for_extend(self)
# 循环内部不再有 req.req_pool_idx 赋值

评论区精华

本 PR 没有 review 评论。唯一评论来自 gemini-code-assist[bot] 的配额警告，与审查无关。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低。变更仅删除了一行冗余赋值和对应的变量绑定。req.req_pool_idx 在分配器内部已经被正确设置，不会因删除该行而丢失。功能回归的可能性很小，但建议验证 HybridReqToTokenPool 的委托路径是否正确设置了该字段。
• 影响：影响范围仅限 ScheduleBatch.prepare_for_extend 方法。代码可维护性提升：职责更清晰，避免未来误解。无用户可见影响，无性能变化。
• 风险标记：缺少测试覆盖

关联脉络

• PR #25728 Pull the max-prefix-len computation into its own helper and rename the matched-token argument: 同属 schedule_batch.py 的重构系列，可能为同一作者或同一重构周期。
• PR #25727 Encapsulate the pending-flush bookkeeping in a small wrapper: 同上，属于调度器组件的封装重构。