Skip to content

Bind temporal ClickHouse summaries in the shared query plan - #568

Closed
zzylol wants to merge 0 commit into
feat/unified-clickhouse-query-planfrom
feat/unified-clickhouse-temporal-bindings
Closed

zzylol wants to merge 0 commit into
feat/unified-clickhouse-query-planfrom
feat/unified-clickhouse-temporal-bindings

Conversation

@zzylol

@zzylol zzylol commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

A ClickHouse query with a timestamp predicate or explicit time range now binds the relevant summary through the authoritative SummaryCatalog and publishes the readout in the shared QueryPlan DAG. This keeps temporal scope in the language-neutral physical plan and preserves SQL identity and external exact fallback.

The change also advances the Planner dependency to the current relational IR stack required by the SQL frontend.

Validation: CARGO_INCREMENTAL=0 cargo check -p control_plane --lib

@zzylol
zzylol force-pushed the feat/unified-clickhouse-temporal-bindings branch 2 times, most recently from 0b30c85 to de4b712 Compare September 10, 2026 14:13
@zzylol
zzylol force-pushed the feat/unified-clickhouse-query-plan branch from 9ec1af9 to 24db7a3 Compare September 10, 2026 14:13
@zzylol
zzylol force-pushed the feat/unified-clickhouse-temporal-bindings branch from de4b712 to c24a40f Compare September 10, 2026 14:16
@zzylol
zzylol force-pushed the feat/unified-clickhouse-query-plan branch from 24db7a3 to b4ec5f9 Compare September 10, 2026 14:16
@zzylol
zzylol force-pushed the feat/unified-clickhouse-temporal-bindings branch from c24a40f to 6ca2965 Compare September 10, 2026 14:29
@zzylol
zzylol force-pushed the feat/unified-clickhouse-query-plan branch 2 times, most recently from 6005c62 to b0f8f4d Compare September 10, 2026 14:30
@zzylol
zzylol force-pushed the feat/unified-clickhouse-temporal-bindings branch from 6ca2965 to 63ada7b Compare September 10, 2026 14:30
@zzylol
zzylol force-pushed the feat/unified-clickhouse-query-plan branch from b0f8f4d to b70791f Compare September 10, 2026 14:35
@zzylol
zzylol force-pushed the feat/unified-clickhouse-temporal-bindings branch from 63ada7b to 744cee3 Compare September 10, 2026 14:35
@zzylol zzylol closed this Sep 10, 2026
@zzylol
zzylol force-pushed the feat/unified-clickhouse-temporal-bindings branch from 744cee3 to b70791f Compare September 10, 2026 14:36
@zzylol

zzylol commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

Superseded: temporal binding is now part of #565, and the Planner relational-IR dependency moved into #569.

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