Skip to content

refactor(control-plane): remove sketch_algebra layer - #467

Merged
zzylol merged 1 commit into
mainfrom
refactor/remove-sketch-algebra
Sep 3, 2026
Merged

zzylol merged 1 commit into
mainfrom
refactor/remove-sketch-algebra

Conversation

@zzylol

@zzylol zzylol commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the top-level control_plane/src/sketch_algebra directory and module
  • place backend-owned adapters under physical/post_asap
  • rename the backend wrapper from L4Plan/physical_expr to PostAsapPlan/deployment_expr
  • document that ASAPPlanner owns post-ASAP planning while the backend owns physical placement, deployment cost, emission, and runtime matching
  • update the repost mock to cover the complete streaming/routing/BackendPlan publication contract

Verification

  • cargo test -p control_plane --lib (688 passed)
  • cargo test -p data_plane query_engines::asap_query_engine --lib (90 passed)
  • cargo test -p data_plane --test e2e_controller_plans_and_backend_serves controller_really_pushes_backend_plan_and_query_really_serves (passed)
  • git diff --check
  • repository search confirms no sketch_algebra, L4Plan, or physical_expr references remain in control_plane/data_plane/README

Stack

Depends on #466.

@zzylol
zzylol force-pushed the refactor/remove-sketch-algebra branch from 1e9e020 to 7eebd8a Compare September 3, 2026 00:45
@zzylol
zzylol changed the base branch from feat/atomic-plan-publication to main September 3, 2026 02:54
@zzylol
zzylol merged commit 7d1a8f8 into main Sep 3, 2026
1 check passed
@zzylol
zzylol deleted the refactor/remove-sketch-algebra branch September 12, 2026 14:48
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