docs(design): close FFB-3/FFB-4/PIP-2 as satisfied by FFB-1 design + PIP-3+5 wire-up#4677
Merged
Merged
Conversation
…PIP-3+5 wire-up Combined closure note covering three trackers whose deliverables have already shipped or were absorbed into existing design surface: - PIP-2 (#2565): the design doc was supposed to describe the token_loop -> ParallelDeltaApplier migration. That document already exists as docs/design/parallel-receive-delta-default-on.md (shipped in PR #4319 scaffold, extended by PR #4666 default-on flip). Marked completed retroactively. - FFB-3 (#2576): conditional task that only activated if FFB-1 chose Option A or B. FFB-1 (PR #4659) chose Option D - barrier baked into finish_file - so every existing caller already gets the barrier without callsite migration. Marked deferred N-A. - FFB-4 (#2577): explicit flush_workers call at the PIP-3 wire-up site. PIP-3 + PIP-5 (PR #4666 merged) routes through finish_file, which already barriers via Option D. No explicit call needed. Marked satisfied-by-design. Note: FFB-2 implementation (PR #4665, in CI) needed a spin-then-yield fix around finish_file's try_unwrap to close a Windows release-race; future work to restructure DecrementGuard would be FFB-5+ if filed. No source changes.
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Combined closure note for three task chains whose deliverables have
already shipped (or were absorbed into existing design surface).
PIP-2 was supposed to produce already exists as
docs/design/parallel-receive-delta-default-on.md, which shippedwith the parallel-receive-delta scaffold (PR feat(engine): SpillableReorderBuffer wiring + parallel receive delta apply (#1884, #1368) #4319) and was extended
by the default-on Path B heuristic (PR perf(transfer): enable parallel receive-delta by default via Path B heuristic (PIP-3 + PIP-5) #4666). No fresh design
artifact required.
choosing Option A or B (explicit
flush_workersprimitive at everycallsite). FFB-1 (PR docs(design): flush_workers barrier API for ParallelDeltaApplier (FFB-1) #4659) chose Option D - the barrier is baked
into
finish_file. FFB-2 (PR feat(engine): add flush_workers/drain_inflight barrier API to ParallelDeltaApplier (FFB-2) #4665, in CI) ships that bundledbarrier. Every existing
finish_filecaller already gets thebarrier without callsite migration.
merged) wired
ParallelDeltaPipelineinto the receiver viaenable_parallel_receive_delta(). The pipeline's chunk handlersreach
apply_one_chunkviaSlotHandledrop, and the receiverfinalises files via
finish_file. Option D's internalflush_workersfires automatically; no explicit call needed at thewire-up site.
Followup note in the doc: the FFB-2 implementation required a
spin-then-yield fix around
finish_file'sArc::try_unwrapto close aWindows release-race (tracked in
project_slothandle_decrementguard_release_race.md). RestructuringDecrementGuardto eliminate the spin would be FFB-5+ if filed.Doc:
docs/design/ffb-3-4-pip-2-closure-2026-05-21.md(145 lines,markdown only, no source changes).
Test plan
closed with the cited rationale.