Skip to content

Split the request-reception loop into smaller phases#25703

Merged
fzyzcjy merged 1 commit into
mainfrom
tom/pr_chain/tom_refactor_202605a/primary/nonmech_scheduler/split-recv-requests
May 19, 2026
Merged

Split the request-reception loop into smaller phases#25703
fzyzcjy merged 1 commit into
mainfrom
tom/pr_chain/tom_refactor_202605a/primary/nonmech_scheduler/split-recv-requests

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented May 19, 2026

Copy link
Copy Markdown
Collaborator

Extract helpers from Scheduler.recv_requests:

  • _pull_raw_reqs: pull raw incoming requests off the IPC PULL socket
  • _broadcast_reqs_across_ranks: replicate the pulled list across ranks
  • _apply_mm_receiver: run the optional multimodal receiver on each req
  • _finalize_shm_features: finalize SHM-backed fields once reqs have
    been broadcast and accepted

Each extracted helper preserves the original body byte-equivalent and is
placed in call order below recv_requests. No behavior change.

Refactor chain ID: extract-recv-requests-helpers


CI States

Latest PR Test (Base): ❌ Run #26070013417
Latest PR Test (Extra): ❌ Blocked -- run-ci is required first.

…ases

Extract helpers from ``Scheduler.recv_requests``:
- ``_pull_raw_reqs``: pull raw incoming requests off the IPC PULL socket
- ``_broadcast_reqs_across_ranks``: replicate the pulled list across ranks
- ``_apply_mm_receiver``: run the optional multimodal receiver on each req
- ``_finalize_shm_features``: finalize SHM-backed fields once reqs have
  been broadcast and accepted

Each extracted helper preserves the original body byte-equivalent and is
placed in call order below ``recv_requests``. No behavior change.

Refactor chain ID: extract-recv-requests-helpers
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@fzyzcjy fzyzcjy merged commit e8e55bb into main May 19, 2026
73 of 84 checks passed
@fzyzcjy fzyzcjy deleted the tom/pr_chain/tom_refactor_202605a/primary/nonmech_scheduler/split-recv-requests branch May 19, 2026 01:12
Shunkangz pushed a commit to Shunkangz/sglang that referenced this pull request May 27, 2026
alphabetc1 pushed a commit to alphabetc1/sglang that referenced this pull request Jun 4, 2026
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.

1 participant