Skip to content

refactor(physical): retire legacy optimizer namespace - #459

Merged
zzylol merged 2 commits into
mainfrom
refactor/remove-legacy-planner
Sep 2, 2026
Merged

zzylol merged 2 commits into
mainfrom
refactor/remove-legacy-planner

Conversation

@zzylol

@zzylol zzylol commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • delete the remaining control_plane::optimizer module and directory
  • move flat workload-to-deployment compatibility into physical::workload_planner
  • move resource/delta/Pareto/TCO calculations into physical::deployment_cost
  • move hysteresis caching into physical::plan_cache
  • rename RulesPlanner, CostModelPlanner, and BaselinePlanner to deployment-oriented terminology

Architectural boundary

ASAPPlanner owns canonical intent algebra, transformations, legal summary candidates, and summary selection. ASAPQuery-backend owns physical CTSA placement, deployment cost reporting, plan caching, emission, and runtime publication. No backend optimizer namespace remains.

The flat workload adapter remains only because the existing /plan and replan wire contracts still accept QueryWorkload; its typed branch delegates summary selection to ASAPPlanner.

MVP verification

  • cargo check -p control_plane -p data_plane
  • cargo test -p control_plane
    • 688 library tests passed
    • 28 API/binary tests passed
    • 0 failures
    • 3 doc tests ignored

This includes physical compilation, OpAMP publication, all five sketch processors, routing, cumulative backend plans, replan, TopK evidence, and MVP workload fixtures.

Stack

Depends on #458.

@zzylol
zzylol changed the base branch from refactor/remove-query-optimizer to main September 2, 2026 17:53
@zzylol zzylol closed this Sep 2, 2026
@zzylol zzylol reopened this Sep 2, 2026
@zzylol
zzylol merged commit 8e3b983 into main Sep 2, 2026
1 check passed
@zzylol
zzylol deleted the refactor/remove-legacy-planner branch September 2, 2026 18:12
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