执行摘要

• 一句话：移除误导性的prompt_tokens_recomputed指标，简化缓存命中统计逻辑。
• 推荐动作：该PR值得精读，尤其是对于关注vLLM指标系统和缓存命中统计的工程师。关键设计决策包括：
  1) 识别并移除误导性指标，避免技术债务；
  2) 简化统计公式，使local_cache_hit和external_kv_transfer的计算更直观；
  3) 与PR #37460的关联展示了指标系统的演进方向。

功能与动机

根据PR描述，vllm:prompt_tokens_recomputed指标的设计初衷是统计因全本地前缀缓存命中而需要丢弃的缓存token数量。然而，在全缓存命中场景下（prompt长度N），系统实际只使用N-1个token，最后一个token需要重新计算以获得logits。但即使在这种情况下，也无法假设最后一个token原本是缓存命中的，因此将其计为“重新计算”是误导性的。该指标是在PR #33290中作为副作用添加的，目的是解释external_kv_transfer指标会包含一个重新计算的token。现在更合理的做法是让external_kv_transfer只统计实际使用的token，而非重新计算的token，这将在PR #37460中实现。由于没有用户依赖此指标，因此直接移除而非经过弃用期。

实现拆解

实现分为三个部分：

1) 在vllm/v1/metrics/loggers.py中移除vllm:prompt_tokens_recomputed指标的计数器定义和记录逻辑；
2) 在vllm/v1/metrics/stats.py中简化PromptTokenStats类，移除recomputed_tokens字段及相关计算逻辑，更新统计公式和update_from_output方法；
3) 在tests/v1/metrics/test_stats.py中更新测试用例，移除对recomputed_tokens的断言，并修正local_cache_hit和external_kv_transfer的预期值。

关键文件：

• vllm/v1/metrics/stats.py（模块 metrics）: 核心变更文件，修改了PromptTokenStats类的统计逻辑，移除recomputed_tokens字段并更新公式和方法。
• vllm/v1/metrics/loggers.py（模块 metrics）: 移除了vllm:prompt_tokens_recomputed指标的定义和记录代码，直接影响指标输出。
• tests/v1/metrics/test_stats.py（模块 tests）: 更新测试用例以反映统计逻辑变更，确保正确性，并修正了external_kv_transfer的测试值。

关键符号：PromptTokenStats.update_from_output

评论区精华

Review讨论较少，仅有的评论来自gemini-code-assist[bot]，指出该PR移除了重新计算token的跟踪和报告，简化了指标系统和PromptTokenStats类，并更新了相关测试，没有提供进一步反馈。orozery直接批准了该PR。没有出现争议或未解决的疑虑。

• 指标移除的合理性与影响 (design): 决定直接移除指标而非弃用，因无用户依赖且设计存在缺陷。

风险与影响

• 风险：技术风险较低：
  1) 指标移除可能导致依赖此指标的监控仪表板或告警规则失效，但PR描述指出没有用户依赖此指标；
  2) 统计逻辑变更可能影响其他依赖PromptTokenStats类的组件，但变更集中在核心公式简化，且测试已更新；
  3) 外部KV传输指标统计逻辑的修正（在测试中体现）可能影响性能分析，但这是向更准确统计的改进。
  • 影响：对用户影响：移除一个可能误导的指标，提升指标系统的清晰度，但可能影响少数内部监控。对系统影响：简化了核心指标统计逻辑，减少计算开销和代码复杂度。对团队影响：需要更新相关文档或内部工具（如果存在），但PR描述表明无用户依赖，因此影响有限。
  • 风险标记：指标移除可能影响监控, 统计逻辑变更

关联脉络

• PR #33290 未知: PR body中提到该指标最初在#33290中添加，作为解释external_kv_transfer指标的副作用。
• PR #37460 未知: PR body中提到external_kv_transfer指标将在此PR中修正，只统计实际使用的token，与本PR的移除决策相关。