Prhub

#45988 [Perf] Remove unused loggers in `reasoning/`

原始 PR 作者 ByteFlowing1337 合并时间 2026-06-19 06:24 文件变更 9 提交数 3 评论 2 代码增减 +0 / -27

执行摘要

移除 reasoning/ 下未使用的日志器

PR 说明:'Remove unused loggers in reasoning/. This makes parsers faster at startup.' 原始代码中每个 parser 文件都创建了日志器,但从未被使用,删除后可减少启动时的初始化开销。

可安全合并。建议类似清理在其他模块推广。

讨论亮点

无实质性讨论。review 仅由 sfeng33 批准,无评论。

实现拆解

  1. 定位 vllm/reasoning/ 目录下所有 parser 文件。
  2. 删除来自 vllm.loggerinit_logger 导入。
  3. 删除对应的 logger = init_logger(__name__) 调用。
    涉及文件共 9 个:deepseek_v3_reasoning_parser.py, ernie45_reasoning_parser.py, granite_reasoning_parser.py, hunyuan_a13b_reasoning_parser.py, identity_reasoning_parser.py, minimax_m2_reasoning_parser.py, mistral_reasoning_parser.py, olmo3_reasoning_parser.py, step3_reasoning_parser.py。每个文件删除 3 行(导入 + 变量赋值)。未涉及测试或配置变更。
文件 模块 状态 重要度
vllm/reasoning/deepseek_v3_reasoning_parser.py 推理解析器 modified 4.35
vllm/reasoning/ernie45_reasoning_parser.py 推理解析器 modified 4.35
vllm/reasoning/granite_reasoning_parser.py 推理解析器 modified 4.35
vllm/reasoning/identity_reasoning_parser.py 推理解析器 modified 4.35
vllm/reasoning/mistral_reasoning_parser.py 推理解析器 modified 4.35

关键源码片段

vllm/reasoning/deepseek_v3_reasoning_parser.py dependency-wiring

删除了未使用的 logger 导入和初始化,是变更的核心文件之一

# 变更后的 deepseek_v3_reasoning_parser.py 顶部
# 删除了未使用的 logger 导入和初始化from collections.abc import Iterable, Sequence
from typing import TYPE_CHECKINGfrom transformers import PreTrainedTokenizerBasefrom vllm.reasoning import ReasoningParser
from vllm.reasoning.deepseek_r1_reasoning_parser import DeepSeekR1ReasoningParserfrom .identity_reasoning_parser import IdentityReasoningParserif TYPE_CHECKING:
    from vllm.entrypoints.openai.chat_completion.protocol import ChatCompletionRequest
    from vllm.entrypoints.openai.engine.protocol import DeltaMessage
    from vllm.entrypoints.openai.responses.protocol import ResponsesRequest# logger = init_logger(__name__) # 这行已被删除class DeepSeekV3ReasoningParser(ReasoningParser):
    ...

评论区精华

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

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

风险与影响

风险极低。删除的日志器确实未在文件其余部分引用,但需确认无外部代码通过模块属性(如 vllm.reasoning.deepseek_v3_reasoning_parser.logger)访问。经检查代码库无此类引用,因此安全。

对用户:启动速度略有提升(毫秒级),运行时无影响。对系统:减少少量模块加载开销。对团队:代码更简洁,减少维护负担。

极低风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论