feat: add build orchestrator#322
Draft
Blackgaurd wants to merge 1 commit into04-17-feat_add_build_worker_that_executes_a_single_jobfrom
Draft
feat: add build orchestrator#322Blackgaurd wants to merge 1 commit into04-17-feat_add_build_worker_that_executes_a_single_jobfrom
Blackgaurd wants to merge 1 commit into04-17-feat_add_build_worker_that_executes_a_single_jobfrom
Conversation
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Apr 17, 2026
0a97d64 to
6fb344e
Compare
4a2ca80 to
f0f679d
Compare
6fb344e to
461b6ea
Compare
f0f679d to
d50c783
Compare
461b6ea to
3820536
Compare
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.

Adds
service/orchestrator.pywithrun_build()that coordinates level-by-level execution of a topological build plan.What changed
service/orchestrator.pywithrun_build(dataset_name, version, start, end)schedule_build()to get aBuildPlan(Kahn's algorithm)execute_job()from the worker modulecancelledevent and raisesRuntimeErrortests/service/test_orchestrator.pywith 6 tests:Why
The orchestrator ties the scheduler and worker together. It owns the "execute the plan" concern, separate from "compute the plan" (scheduler) and "build one dataset" (worker).
Benefit
run_build()is the drop-in replacement for_build_recursive(). PR 6 will wire it intobuild_dataset()and delete the old code.🤖 Generated with Claude Code