refactor: move the unified frontends and physical planner to the crate roots

[zzylol]

Why

After #542 the planner calls the unified modules through their temporary unified paths, and the replaced legacy frontend modules are dead. This PR gives the unified code its final paths, so the crates read as they will stay.

What

Moves, import updates and the documentation/tooling that names the moved APIs. No planner or execution behavior change.

• asap_frontend_{sql,promql,metricsql}::unified become the crate roots; the replaced legacy frontend modules and their tests are deleted.
• unified_physical_planner / unified_sources become physical_planner / sources; expressions/unified_planner.rs becomes the private expressions/planner.rs (re-exported as expressions::CompiledExpression), and Expression::UnifiedPlanner becomes Expression::Planner.
• The unified_* tests take the names of the tests they replace (sql_lowering.rs, promql_lowering.rs, promql_conformance.rs, histogram_metadata.rs, scalar_design.rs, support.rs, common/, precompute.rs, node_ids.rs; MetricsQL lowering_semantics.rs).
• crates/asap-physical-operators/README.md is corrected: physical_planner::compile takes a Planner PhysicalASAPDAG (not PostAsapDAG), the separate unified_physical_planner paragraph is folded in, and the module list says evaluation instead of readout.
• tools/verify_metricsql_external_consumer.sh calls lower_metricsql with the new ingestion-interval argument and removes its temporary crate on exit.
• Tests: the moved files use the stack's wording ("tree" → "DAG", "readout" → "evaluation"); the MetricsQL legacy lowering test is ported to the unified API with the ingestion-interval parameter; the duplicate default_rollup-without-range test is removed (rollups_without_an_explicit_range_are_rejected covers it).

How

The unified modules replace the legacy ones at the same paths: the legacy frontend modules and their tests are deleted, and the unified files are moved into their place with imports updated. In the merged diff, the physical planner, sources and test files show as renames; the frontend crates show as edits of the root modules plus deletion of unified/. The external-consumer script, the README and test wording are fixed as listed above. No stage logic changes; this only gives the Stage 0 frontends and the physical planner their final paths.

Before this PR

The unified code lives at …::unified, unified_physical_planner, unified_sources, next to dead legacy frontend modules whose root functions still return QueryExpr.

After this PR

One SQL, PromQL and MetricsQL frontend at each crate root, and one physical_planner. Callers import from the roots. Public API changes at the crate roots (the root functions now have the former unified signatures):

• SQL: lower_sql, lower_sql_dialect return Rc<OperatorNode>; lower_sql_batch returns Vec<Result<Rc<OperatorNode>, SqlError>> (previously QueryExpr).
• PromQL: lower_promql_workload and lower_promql_workload_with_histograms return Vec<Rc<OperatorNode>>; lower_promql_query_workload and lower_promql_query_workload_with_histograms (returning Vec<QueryRoot>) are new at the root.
• MetricsQL: lower_metricsql(query, accuracy, ingestion_interval: Duration) returns Rc<OperatorNode> (previously (query, accuracy) → QueryExpr); lower_metricsql_query (→ QueryRoot) and lower_metricsql_query_workload (→ Vec<QueryRoot>) are new at the root.
• Physical operators: unified_physical_planner → physical_planner, unified_sources → sources, Expression::UnifiedPlanner → Expression::Planner.
• The unified modules no longer exist.

Evidence

Not applicable (no behavior change).

Verification

• Unit, integration and end-to-end tests: unchanged apart from paths, wording and the one removed duplicate.
• Other checks on the PR head: cargo fmt --check; workspace all-target Clippy with -D warnings; cargo test --workspace (1,726 passed, 0 failed, 2 ignored); dag-viewer test_render (31 passed); tools/verify_metricsql_external_consumer.sh OK.

Architectural decisions

• A separate move PR keeps 15k lines of moves out of the cutover (refactor(planner): plan the workload on the unified operator IR #542) and the IR removal (refactor(ir): remove the legacy QueryExpr and SummaryExpr IRs; move docs and DAG viewer to the unified IR #543).

Limitations and follow-up

• refactor(ir): remove the legacy QueryExpr and SummaryExpr IRs; move docs and DAG viewer to the unified IR #543 removes the legacy QueryExpr / SummaryExpr IRs.

Stack: tracker #528 · main → #542 → #665 → #666 → #543 → #667 → #561.

Human review — do not complete with an agent

• The MVP boundary is correct.
• New conceptual layers or public interfaces are necessary.
• The before/after description matches the intended product behavior.
• Human reviewer:
• Decision and rationale:

🤖 Generated with Claude Code

https://claude.ai/code/session_01W7qG9aFyPij5uWsyAJCxDW