执行摘要

• 一句话：移除 MooncakeStore 中废弃的 discard_partial_chunks 参数
• 推荐动作：值得合并。作为死代码清理，变更安全且测试充分。推荐花 1-2 分钟阅读以了解 MooncakeStore 的块对齐逻辑。

功能与动机

作为 #43494 的收尾工作，该 PR 移除了 from_request_tracker 方法中不再使用的 discard_partial_chunks 参数及其不可达的 else 分支。PR body 明确指出“No behavior change”且所有测试均通过。

实现拆解

1. 删除参数：在 vllm/distributed/kv_transfer/kv_connector/v1/mooncake/store/data.py 的 ReqMeta.from_request_tracker 方法签章中移除 discard_partial_chunks: bool = True。
2. 简化计算：删除 chunk_boundary 和 num_tokens_to_save 的 if/else 条件分支，始终使用原 discard_partial_chunks=True 的逻辑（即按块对齐）。
3. 测试验证：在分支上运行相关单元测试（12 + 67 项）均通过，ruff 检查干净。

关键文件：

• vllm/distributed/kv_transfer/kv_connector/v1/mooncake/store/data.py（模块 KV 连接器；类别 source；类型 core-logic；符号 ReqMeta.from_request_tracker）: 唯一变更文件，移除 discard_partial_chunks 参数及条件分支，简化 chunk_boundary 和 num_tokens_to_save 计算。

关键符号：ReqMeta.from_request_tracker

关键源码片段

vllm/distributed/kv_transfer/kv_connector/v1/mooncake/store/data.py

唯一变更文件，移除 discard_partial_chunks 参数及条件分支，简化 chunk_boundary 和 num_tokens_to_save 计算。

# vllm/distributed/kv_transfer/kv_connector/v1/mooncake/store/data.py
# 变更后：移除了 discard_partial_chunks 参数，始终使用块对齐逻辑
​
@staticmethod
def from_request_tracker(
    tracker: RequestTracker,
    block_size: int,
    load_spec: LoadSpec | None = None,
    skip_save: bool | None = False,
    block_hashes: list[BlockHash] | None = None,
    is_last_chunk: bool | None = None,
    original_block_size: int | None = None,
) -> "ReqMeta | None":
    """Create ReqMeta from a RequestTracker."""
    if block_hashes is None:
        block_hashes = []
    input_token_len = tracker.token_len
​
    # 删除 discard_partial_chunks 后，始终使用以下块对齐逻辑
    chunk_boundary = cdiv(tracker.num_saved_tokens + 1, block_size) * block_size
    num_tokens_to_save = input_token_len // block_size * block_size
​
    skip_save = skip_save or num_tokens_to_save < chunk_boundary
​
    # ReqMeta 不能同时包含 save 和 load
    if load_spec is not None and load_spec.can_load:
        skip_save = True
    if skip_save and load_spec is None:
        return None
​
    if not skip_save:
        tracker.num_saved_tokens = num_tokens_to_save
​
    # ... 其余代码不变

评论区精华

PR 获得 3 个 approve（ivanium, Dao007forever, njhill），无 review 评论。gemini-code-assist 的自动评论确认了变更内容。作者在 PR body 中主动讨论了是否应推迟合并，但最终决定直接清理。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低。变更仅移除已废弃的参数和死分支，唯一保留的分支是原 discard_partial_chunks=True 的路径，与 #43494 合并后的行为完全一致。lmcache_integration 中的同名方法属不同模块，不受影响。
• 影响：影响范围限于 MooncakeStore 内部，仅修改一个文件，+2/-11 行。对用户无感知，对系统无性能影响。清理代码后降低了未来维护成本。
• 风险标记：低风险死代码清理

关联脉络

• PR #43494 Keep MooncakeStore full hits block-aligned: 本 PR 是该 PR 的后续清理，移除了该 PR 保留的残留参数。