Skip to content

[feature] Coordinate GammaLake predicate pushdown - #22

Merged
gauglertodd merged 1 commit into
mainfrom
parity/18-predicate-pushdown
Sep 1, 2026
Merged

[feature] Coordinate GammaLake predicate pushdown#22
gauglertodd merged 1 commit into
mainfrom
parity/18-predicate-pushdown

Conversation

@gauglertodd

@gauglertodd gauglertodd commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Coordinates predicate pushdown through the polars-io-tools adapter so filters are applied as early as correctness permits. Canonical-key joins preserve alignment; runtime/as-of paths retain neighbor context, and timezone-aware temporal predicates bypass the upstream 0.2.1 naive-datetime translation.

Stacked PR 18/19. Base: parity/17-lazy-multi-source.

@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from 2bf0c09 to ec6d78f Compare August 31, 2026 16:18
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 6a5e485 to 5d22a56 Compare August 31, 2026 16:18
@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from ec6d78f to f1444ab Compare August 31, 2026 19:04
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 5d22a56 to a6715ae Compare August 31, 2026 19:04
@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from f1444ab to 6c11f51 Compare August 31, 2026 20:29
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from a6715ae to cd80430 Compare August 31, 2026 20:29
@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from 6c11f51 to ac94079 Compare August 31, 2026 20:42
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from cd80430 to faa4877 Compare August 31, 2026 20:42
@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from ac94079 to 8d200ec Compare August 31, 2026 20:53
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from faa4877 to 371650a Compare August 31, 2026 20:53
@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from 8d200ec to 63ca139 Compare September 1, 2026 14:48
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 371650a to 2665371 Compare September 1, 2026 14:48
@gauglertodd
gauglertodd changed the base branch from parity/17-lazy-multi-source to parity/pr-stack-refresh-20260901 September 1, 2026 15:00
@gauglertodd
gauglertodd changed the base branch from parity/pr-stack-refresh-20260901 to parity/17-lazy-multi-source September 1, 2026 15:00
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 2665371 to 488cceb Compare September 1, 2026 15:00
@gauglertodd
gauglertodd changed the base branch from parity/17-lazy-multi-source to parity/pr-stack-refresh-20260901b September 1, 2026 15:08
@gauglertodd
gauglertodd changed the base branch from parity/pr-stack-refresh-20260901b to parity/17-lazy-multi-source September 1, 2026 15:08
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 488cceb to 2f4a9ff Compare September 1, 2026 15:10
@gauglertodd
gauglertodd changed the base branch from parity/17-lazy-multi-source to parity/pr-stack-refresh-20260901c September 1, 2026 15:17
@gauglertodd
gauglertodd changed the base branch from parity/pr-stack-refresh-20260901c to parity/17-lazy-multi-source September 1, 2026 15:17
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 2f4a9ff to cf6671f Compare September 1, 2026 15:19
@gauglertodd
gauglertodd changed the base branch from parity/17-lazy-multi-source to parity/pr-stack-refresh-20260901d September 1, 2026 15:20
@gauglertodd
gauglertodd changed the base branch from parity/pr-stack-refresh-20260901d to parity/17-lazy-multi-source September 1, 2026 15:20
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from cf6671f to 2eaa079 Compare September 1, 2026 15:22
@gauglertodd
gauglertodd force-pushed the parity/17-lazy-multi-source branch from 5397a96 to d517774 Compare September 1, 2026 15:45
@gauglertodd
gauglertodd changed the base branch from parity/17-lazy-multi-source to parity/pr-iotools-refresh-20260901 September 1, 2026 15:45
@gauglertodd
gauglertodd changed the base branch from parity/pr-iotools-refresh-20260901 to parity/17-lazy-multi-source September 1, 2026 15:45
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 2eaa079 to 8ceb4f2 Compare September 1, 2026 15:46
Base automatically changed from parity/17-lazy-multi-source to main September 1, 2026 16:22
Tracked by CCRT-7150.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: gauglertodd <todd.gaugler@cubistsystematic.com>
@gauglertodd
gauglertodd force-pushed the parity/18-predicate-pushdown branch from 8ceb4f2 to f81a729 Compare September 1, 2026 16:26
@gauglertodd
gauglertodd changed the base branch from main to parity/pr22-refresh-20260901 September 1, 2026 16:26
@gauglertodd
gauglertodd changed the base branch from parity/pr22-refresh-20260901 to main September 1, 2026 16:26
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Test Results

138 tests  +13   138 ✅ +13   42s ⏱️ ±0s
  1 suites ± 0     0 💤 ± 0 
  1 files   ± 0     0 ❌ ± 0 

Results for commit f81a729. ± Comparison against base commit ce0265a.

This pull request removes 1 and adds 14 tests. Note that renamed tests count towards both.
gammalake.tests.test_multi_source ‑ test_rejects_mismatched_filtered_heights
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_as_of_features_compute_before_lazy_filter
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_feature_filter_matches_filtering_unmaterialized_result
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_mixed_dense_sparse_read_aligns_values_to_keys
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_multi_source_filter_plan_and_exact_equivalence
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_multi_sparse_read_aligns_when_tails_differ
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_multi_table_read_preserves_horizontal_alignment[False]
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_multi_table_read_preserves_horizontal_alignment[True]
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_non_leading_primary_sort_key_preserves_alignment_and_bounds
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_projection_only_aggregation_remains_correct
gammalake.tests.test_gamma_feature_lake.TestGammaFeatureLake ‑ test_runtime_features_compute_before_lazy_filter
…

@gauglertodd
gauglertodd merged commit 1584f32 into main Sep 1, 2026
1 check passed
@gauglertodd
gauglertodd deleted the parity/18-predicate-pushdown branch September 1, 2026 16:33
gauglertodd added a commit that referenced this pull request Sep 2, 2026
Signed-off-by: gauglertodd <16946312+gauglertodd@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
gauglertodd added a commit that referenced this pull request Sep 2, 2026
Signed-off-by: gauglertodd <16946312+gauglertodd@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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