Repository navigation
Domain uses AST analysis, not the SQL translator - #134
austin-barrington wants to merge 1 commit into
Conversation
timeseriesql::analysis holds the AST facts that hold whatever the backend: select_output_field_name (with default_field_alias / influxql_name), select_has_true_aggregate and the aggregate/window-transform predicates, SelectorKind, MultiSelector, BUCKET_COL and TIME_NS_COL. to_clickhouse re-exports them, so the translator is unchanged; domain::rollup, sharding::query_merge and sharding::multi_selector import the analysis module instead of to_clickhouse.
PR SummaryLow Risk Overview Domain code in The moved logic is lifted out of Reviewed by Cursor Bugbot for commit 88a2840. Configure here. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: bb52c930-66ad-405e-94ef-5139fa612dcb) |
Stacked on #133. Implements
docs/superpowers/plans/2026-11-03-query-analysis-module.md.timeseriesql::analysisnow holds the pure InfluxQL AST facts the domain needs:domain::rollup,domain::sharding::query_mergeanddomain::sharding::multi_selectornow import from it instead ofto_clickhouse. The translator re-exports the moved items, so the generated SQL is unchanged.The move is pure: a reviewer compared every moved body line by line, and only visibility changed. New tests pin output names, that window transforms are not aggregates, and the merge column literals.
Gate: fmt,
clippy -D warnings, 883 lib tests, 373 integration tests and 54 CLI tests pass.