Skip to content

perf: transcode Python request plane directly (cherry-pick of #11104) - #11283

Open
nv-yna wants to merge 1 commit into
feat/deepseek_v4_aafrom
yna/cherry-pick-11104-python-request-plane
Open

perf: transcode Python request plane directly (cherry-pick of #11104)#11283
nv-yna wants to merge 1 commit into
feat/deepseek_v4_aafrom
yna/cherry-pick-11104-python-request-plane

Conversation

@nv-yna

@nv-yna nv-yna commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Cherry-pick of #11104 ("perf: transcode Python request plane directly") onto feat/deepseek_v4_aa.

Mirrors the methodology from #11138 (cherry-pick of #10437 onto this same branch).

Changes included (verbatim from #11104)

  • lib/bindings/python/rust/python_payload.rs — new Python-native msgpack payload encoder
  • lib/bindings/python/rust/engine.rs — use Python payload codec directly (avoid JSON round-trip)
  • lib/bindings/python/rust/lib.rs — wiring
  • lib/runtime/src/pipeline/network.rsRequestPlanePayloadCodec made pub; new default is Msgpack
  • lib/runtime/src/pipeline/network/ingress/push_handler.rs — serialization error handling
  • lib/runtime/src/metrics/prometheus_names.rsSERIALIZATION error variant added
  • New tests: test_request_plane_python_payload.py, bidirectional_e2e_json.rs
  • tests/frontend/realtime_echo_worker.py — updated for codec changes

Divergence / conflict resolution

feat/deepseek_v4_aa is ~236 commits behind main. The cherry-pick applied cleanly with no conflicts — git auto-resolved all 5 touched files (Cargo.lock, Cargo.toml, lib.rs, prometheus_names.rs, network.rs) because #10437 (the prerequisite codec PR) was already on feat as of #11138.

No ambient-main symbols were introduced. Diff counts exactly match the source commit: 13 files, +1349/-270.

Build result

PASS — both mandatory checks clean:

  • cargo check -p dynamo-runtime --testsFinished dev profile in 23.77s
  • cargo check (Python bindings crate) → Finished dev profile in 54.40s

Do NOT merge

Leave merging to @yna — rebase-merge preferred (same as #11138).

Automated cherry-pick triggered by merge of #11104.

Signed-off-by: jthomson04 <jwillthomson19@gmail.com>
(cherry picked from commit 21043cd)
Signed-off-by: Yuewei Na <nv-yna@users.noreply.github.com>
@nv-yna
nv-yna requested review from a team as code owners July 6, 2026 22:09
@nv-yna
nv-yna requested review from a team and jthomson04 July 6, 2026 22:09
@github-actions github-actions Bot added the perf label Jul 6, 2026
@datadog-official

datadog-official Bot commented Jul 6, 2026

Copy link
Copy Markdown

Pipelines

⚠️ Warnings

🚦 8 Pipeline jobs failed

Docs link check | lychee   View in Datadog   GitHub Actions

PR | backend-status-check   View in Datadog   GitHub Actions

PR | dynamo-status-check   View in Datadog   GitHub Actions

View all 8 failed jobs.

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 9d3b1ac | Docs | Give us feedback!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant