Skip to content

Comments

Fix view rewrite for delta tables#3801

Merged
joshua-spacetime merged 1 commit intomasterfrom
joshua/fix/view-rewite-for-deltas
Dec 1, 2025
Merged

Fix view rewrite for delta tables#3801
joshua-spacetime merged 1 commit intomasterfrom
joshua/fix/view-rewite-for-deltas

Conversation

@joshua-spacetime
Copy link
Collaborator

Description of Changes

Applies the same implicit filter for views to delta tables that we already did for physical tables.

Removes a single condition scan.delta.is_none() from the already existing rewrite rule.

API and ABI breaking changes

None

Expected complexity level and risk

0

Testing

  • Rust sdk test

@joshua-spacetime joshua-spacetime changed the title Fix view rewrite for delta plans Fix view rewrite for delta tables Dec 1, 2025
@joshua-spacetime joshua-spacetime added the bugfix Fixes something that was expected to work differently label Dec 1, 2025
Copy link
Contributor

@mamcx mamcx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@joshua-spacetime joshua-spacetime added this pull request to the merge queue Dec 1, 2025
Merged via the queue into master with commit c49eea3 Dec 1, 2025
29 checks passed
@joshua-spacetime joshua-spacetime deleted the joshua/fix/view-rewite-for-deltas branch December 1, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes something that was expected to work differently release-1.11.0 release-next

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants