Skip to content

Add rewrite context to MV cost function#38

Merged
xudong963 merged 2 commits into
branch-52from
xudong963/rewrite-context
Apr 7, 2026
Merged

Add rewrite context to MV cost function#38
xudong963 merged 2 commits into
branch-52from
xudong963/rewrite-context

Conversation

@xudong963
Copy link
Copy Markdown
Collaborator

Summary

  • add rewrite context metadata for candidate selection in view exploitation
  • pass source table refs from logical rewrite into OneOf and OneOfExec
  • extend CostFn to receive cost context and add tests covering context propagation

Validation

  • cargo fmt --all
  • CARGO_INCREMENTAL=0 cargo check --tests
  • cargo test --lib rewrite::exploitation -- --nocapture (linking failed due to local disk space: errno=28)

Copy link
Copy Markdown
Collaborator

@zhuqi-lucas zhuqi-lucas left a comment

Choose a reason for hiding this comment

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

LGTM

@xudong963 xudong963 merged commit c2f4eb3 into branch-52 Apr 7, 2026
8 checks passed
@xudong963 xudong963 deleted the xudong963/rewrite-context branch April 7, 2026 07:45
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.

2 participants