Skip to content

refactor: delete unused precompute coordinator and series buffer - #790

Merged
zzylol merged 1 commit into
mainfrom
refactor/remove-dead-precompute
Sep 29, 2026
Merged

zzylol merged 1 commit into
mainfrom
refactor/remove-dead-precompute

Conversation

@zzylol

@zzylol zzylol commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Why

Step toward DAG-only precompute execution: remove precompute modules that no
production path uses, so the remaining engine code reflects what actually runs.

What

Delete three modules (1.8k lines) that are referenced only by each other and
by precompute_engine/mod.rs:

  • multisource_coordinator.rs (1104 lines)
  • coordination_checkpoint.rs (548 lines)
  • series_buffer.rs (163 lines)

precompute_engine_design_doc.md drops the SeriesBuffer section (§3.5,
later sections renumbered), its SeriesState field and the module-table and
unit-test mentions.

Before this PR

$ git grep -l 'multisource_coordinator\|coordination_checkpoint\|SeriesBuffer' -- '*.rs'
data_plane/src/precompute_engine/coordination_checkpoint.rs
data_plane/src/precompute_engine/mod.rs
data_plane/src/precompute_engine/multisource_coordinator.rs
data_plane/src/precompute_engine/series_buffer.rs

After this PR

No references remain. No behavior change.

Not in scope

Validation

cargo clippy --workspace --all-targets --locked -- -D warnings and
cargo fmt --check pass; cargo test --workspace --locked --lib: 1449 passed.

🤖 Generated with Claude Code

`multisource_coordinator`, `coordination_checkpoint` and `series_buffer`
have no callers outside themselves and `mod.rs`; production precompute
does not construct them. Remove them (1.8k lines) and their design-doc
sections.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zzylol
zzylol merged commit 54067d4 into main Sep 29, 2026
1 check passed
@zzylol
zzylol deleted the refactor/remove-dead-precompute branch September 29, 2026 20:39
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