执行摘要

• 一句话：NixlPushConnector 禁止双向传输模式
• 推荐动作：该 PR 是防御性编程的典型示例，值得快速合入。对于维护 KV Connector 模块的团队，建议检查其他 connector 是否有类似缺失的配置校验。

功能与动机

Bidirectional KV transfer 与 Push 模型不兼容，但缺少运行时校验，可能导致静默错误或未定义行为。PR 旨在强制阻止这种非法配置。

实现拆解

1. 在 NixlPushConnectorScheduler.__init__ 增加守卫检查：在调用父类初始化后，立即检查 self.is_bidirectional_kv_xfer_enabled。若为 True，则抛出 NotImplementedError 并给出明确提示信息。
2. 无其他文件变更：仅修改 vllm/distributed/kv_transfer/kv_connector/v1/nixl/push_scheduler.py 一个文件，增加 4 行代码，无删除。

关键文件：

• vllm/distributed/kv_transfer/kv_connector/v1/nixl/push_scheduler.py（模块 分布式；类别 source；类型 core-logic；符号 NixlPushConnectorScheduler.init）: 核心变更文件，在构造函数中增加配置守卫校验

关键符号：NixlPushConnectorScheduler.init

关键源码片段

vllm/distributed/kv_transfer/kv_connector/v1/nixl/push_scheduler.py

核心变更文件，在构造函数中增加配置守卫校验

class NixlPushConnectorScheduler(NixlBaseConnectorScheduler):
    def __init__(
        self,
        vllm_config: VllmConfig,
        engine_id: str,
        kv_cache_config: KVCacheConfig,
    ):
        super().__init__(vllm_config, engine_id, kv_cache_config)
        # 守卫：Push 模式不支持双向 KV 传输
        if self.is_bidirectional_kv_xfer_enabled:
            raise NotImplementedError(
                "Bidirectional KV transfer is not supported for NIXL push connector."
            )
        # 其余初始化逻辑 ...

评论区精华

无 review 评论。PR 由 DarkLight1337 直接 approve。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低。新增的守卫仅在配置不当（同时启用 push 和双向传输）时触发，不会影响已有功能。若某些用户依赖此非法组合，会立即因异常而失败，但属于预期行为。
• 影响：影响范围极小：仅对使用 NixlPushConnector 且同时启用 bidirectional KV transfer 的配置生效。此前这些配置可能静默工作但结果不可预测，现在将被显式拒绝。
• 风险标记：暂无

关联脉络

• PR #45053 [KV Offload] Replace OffloadingHandler with OffloadingWorker: 同为 KV Connector 模块的架构变更，涉及 Worker/Scheduler 角色划分。