执行摘要

• 一句话：重命名会话计数函数并使用辅助函数简化流式会话检测。
• 推荐动作：建议快速浏览此 PR，重点关注命名改进和辅助函数的使用，以学习代码风格优化技巧。对于深入了解流式会话内存管理机制的工程师，可结合 #22651 和 #22753 阅读。

功能与动机

PR body 指出，sessions in session_controller.sessions are always alive (removed on close), so 'alive' adds no information。同时，为了减少代码重复和提高可读性，使用现有的 _is_streaming helper 替换内联的复杂检测逻辑。这是 streaming session memory accounting fix series (#22651) 的一部分，stacked on #22753。

实现拆解

1. 重命名核心函数：在 python/sglang/srt/managers/scheduler_runtime_checker_mixin.py 中，将 _alive_streaming_session_count 重命名为 _streaming_session_count，因为会话在 controller 中总是活跃的，无需额外标注。
2. 导入并使用辅助函数：在 python/sglang/srt/mem_cache/common.py 中，从 session_aware_cache 导入 _is_streaming，并替换 release_kv_cache 函数中内联的 getattr(req, "session", None) is not None and req.session.streaming 检测逻辑，提高代码复用性。
3. 更新调用点：在 python/sglang/srt/observability/scheduler_metrics_mixin.py 和 scheduler_runtime_checker_mixin.py 的 _maybe_log_idle_metrics 方法中，更新对重命名函数的调用，确保统计正确。
4. 修复监控文档：在 python/sglang/srt/observability/metrics_collector.py 中，将 Prometheus Gauge 的文档从 "active streaming sessions" 改为 "streaming sessions"，以匹配函数名更改。
5. 清理注释：在 python/sglang/srt/managers/session_controller.py 中，将注释 "No active request" 改为 "No owning request"，使用更准确的术语。

关键文件：

• python/sglang/srt/managers/scheduler_runtime_checker_mixin.py（模块 调度器；类别 source；类型 core-logic；符号 _alive_streaming_session_count, _streaming_session_count）: 定义了流式会话计数的核心函数，重命名影响多个调用点，是 PR 的关键变更入口。
• python/sglang/srt/mem_cache/common.py（模块 缓存层；类别 source；类型 dependency-wiring）: 修改导入并使用 _is_streaming helper 简化流式会话检测逻辑，减少代码重复。
• python/sglang/srt/observability/scheduler_metrics_mixin.py（模块 可观测性；类别 source；类型 core-logic）: 更新调度器统计中流式会话计数的调用点，确保指标收集正确。
• python/sglang/srt/observability/metrics_collector.py（模块 可观测性；类别 source；类型 documentation）: 修复 Prometheus Gauge 文档，从 'active streaming sessions' 改为 'streaming sessions'，以匹配函数名更改。
• python/sglang/srt/managers/session_controller.py（模块 会话管理；类别 source；类型 style）: 清理注释，将 'No active request' 改为 'No owning request'，使用更准确的术语。

关键符号：_streaming_session_count, _is_streaming

评论区精华

无实质性 review 讨论，PR 直接合并，表明变更被视为低风险且一致通过。

• 暂无高价值评论线程

风险与影响

• 风险：风险较低，主要涉及符号重命名：
  • 符号重命名风险：所有调用 _alive_streaming_session_count 的地方都已更新为新名 _streaming_session_count，但需确保无遗漏，否则可能导致运行时错误。
  • 依赖更新风险：在 mem_cache/common.py 中导入 _is_streaming helper，若该 helper 未来变更或不存在，可能影响 KV 缓存释放逻辑。
  • 文档一致性风险：Prometheus 指标文档更新需与实际功能一致，避免监控误解。
  • 影响：
    • 用户影响：无直接影响，为内部代码重构。
    • 系统影响：提高代码可读性和一致性，减少重复代码，有利于长期维护。
    • 团队影响：作为内存核算修复系列的一环，增强流式会话处理的可靠性，便于后续调试和扩展。
    • 风险标记：符号重命名风险, 依赖更新风险

关联脉络

• PR #22651 enable streaming session retract tests: 本 PR 是该内存核算修复系列的一部分，旨在确保流式会话测试通过。
• PR #22753 Fix streaming session busy-check double-counting via active_pool_idxs: PR body 指出本 PR stacked on #22753，两者共同修复流式会话内存核算问题。