Skip to content

feat: Complete P2PConnector implementation for high-performance PD Disaggregation#997

Open
ZhihanYan wants to merge 1 commit into
mainfrom
feature/p2p-connector-0513
Open

feat: Complete P2PConnector implementation for high-performance PD Disaggregation#997
ZhihanYan wants to merge 1 commit into
mainfrom
feature/p2p-connector-0513

Conversation

@ZhihanYan
Copy link
Copy Markdown
Collaborator

Protocol Definition: Added TransferService Proto definitions to support both RDMA and TCP transport protocols.
Decode Side Implementation:
Implemented DecodeRpcServerNew2 to coordinate asynchronous Prefill calls and execute the Decode phase.
Prefill Side Implementation:
Implemented PrefillRpcServerNew2 to handle incoming Prefill requests and manage P2P cache loading.
Client Component:
Introduced PrefillServerCaller to support asynchronous Prefill RPC calls with connection pooling capabilities.
Initialization & API:
Enabled P2PConnector initialization logic.
Exposed the loadCacheSync interface for synchronous cache loading operations.

@ZhihanYan ZhihanYan requested a review from LLLLKKKK as a code owner May 13, 2026 01:56
@LLLLKKKK
Copy link
Copy Markdown
Collaborator

AI Code Review - PR #997

Status: LGTM

Summary: P0/0 · P1/0 · P2/0 · P3/0

lgtm ready to ci

Checklist Violations (1 fail / 56 total)

General Principles Checklist

  • [6.1] Quality — Mega-PR 已拆分为独立变更 → checklist-only
    PR 涉及 105 个文件,覆盖 P2P connector、RPC、Python routing、smoke/golden。范围偏大,但变更围绕 decode_entrance 同一能力,本轮不升级为单独 issue。

Strengths

  • decode_entrance 的 P2P side-channel、StartLoad 兼容解析、Python 路由和 smoke 覆盖形成了较完整的端到端链路。
  • 新增 RPC/取消/错误传播测试覆盖了多个失败路径,降低了新 prefill 调用链的回归风险。

@wht21
Copy link
Copy Markdown
Collaborator

wht21 commented May 13, 2026

internal source has been updated, please review the changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants