Skip to content

docs: address the sampling pools design review - #379

Open
ShubyM wants to merge 1 commit into
openrl-dev:mainfrom
ShubyM:docs/pools-design-review
Open

ShubyM wants to merge 1 commit into
openrl-dev:mainfrom
ShubyM:docs/pools-design-review

Conversation

@ShubyM

@ShubyM ShubyM commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Addresses Hoda's comments on the previous design (#324) and matches the doc to the code in #357 and #378, plus the fairness and failures PRs that follow.

  • Shared LoRA pools: a non-exclusive LoRA pool is always routed by default, even at one replica. When tenants ask for different sampler_replicas, the pool grows to the largest and never shrinks. Placement is best effort, and node role labels keep sampler growth off trainer GPUs.
  • Fairness: the dispatcher sends each pool's requests round-robin by job, so one job's big batch doesn't hold up the others.
  • Revision switching: only FFT pools switch revisions, and each belongs to one job, so a pool switches once its current revision drains. An old delta no replica can apply fails right away.
  • Missing pools: a missing pool no longer strands requests. The API server recreates it for a live job or fails the job's requests.

New open item: a pool's replicas can land on different GPU types, since placement asks only for memory. Pinning later replicas to the first replica's product is a follow-up.

Shared LoRA pools are always routed and grow to the largest sampler_replicas any job asks for. The dispatcher sends
round-robin by job, only FFT pools switch revisions and do so once the current revision drains, and the API server
recreates a missing pool for a live job or fails its requests. Adds the open item that a pool's replicas can land on
different GPU types.
@ShubyM
ShubyM marked this pull request as ready for review October 26, 2026 17:35

This branch has not been deployed

No deployments
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