执行摘要
移除 reasoning/ 下未使用的日志器
PR 说明:'Remove unused loggers in reasoning/. This makes parsers faster at startup.' 原始代码中每个 parser 文件都创建了日志器,但从未被使用,删除后可减少启动时的初始化开销。
可安全合并。建议类似清理在其他模块推广。
无实质性讨论。review 仅由 sfeng33 批准,无评论。
PR 说明:'Remove unused loggers in reasoning/. This makes parsers faster at startup.' 原始代码中每个 parser 文件都创建了日志器,但从未被使用,删除后可减少启动时的初始化开销。
可安全合并。建议类似清理在其他模块推广。
无实质性讨论。review 仅由 sfeng33 批准,无评论。
vllm/reasoning/ 目录下所有 parser 文件。vllm.logger 的 init_logger 导入。logger = init_logger(__name__) 调用。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_CHECKING
from transformers import PreTrainedTokenizerBase
from vllm.reasoning import ReasoningParser
from vllm.reasoning.deepseek_r1_reasoning_parser import DeepSeekR1ReasoningParser
from .identity_reasoning_parser import IdentityReasoningParser
if 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 链接,后续同步到相关引用后会出现在这里。
参与讨论