Skip to content

refactor: move PruningPredicate into its own module #16587

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
adriangb merged 2 commits into from
Jun 27, 2025
Merged

refactor: move PruningPredicate into its own module #16587

adriangb merged 2 commits into from
Jun 27, 2025

Conversation

@adriangb
Copy link
Contributor

@adriangb adriangb commented Jun 27, 2025

Part of #16424

@adriangb adriangb force-pushed the move-predicate-mod branch from 7ea93ab to 6ed4538 Compare June 27, 2025 15:06
@adriangb adriangb force-pushed the move-predicate-mod branch from 6ed4538 to 76e66ce Compare June 27, 2025 15:06
@adriangb
Copy link
Contributor Author

adriangb commented Jun 27, 2025

This confirms there are no actual changes:

git show $(git rev-parse main):datafusion/pruning/src/lib.rs > /tmp/old_lib.rs && git diff --no-index /tmp/old_lib.rs datafusion/pruning/src/pruning_predicate.rs

@adriangb adriangb mentioned this pull request Jun 27, 2025
Merged
alamb
alamb approved these changes Jun 27, 2025
View reviewed changes
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you

@adriangb adriangb merged commit 586a88c into apache:main Jun 27, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alamb alamb alamb approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adriangb @alamb