Skip to content

Conversation

@timsaucer
Copy link
Member

@timsaucer timsaucer commented Nov 7, 2025

Which issue does this PR close?

Rationale for this change

Our python downstream library no longer uses the feature pyarrow in this repository #1000. As such, this feature is no longer needed.

What changes are included in this PR?

Code removal

Are these changes tested?

Tested against datafusion-python repo. No build issues.

Are there any user-facing changes?

If any user needs this code, they will need to migrate to the datafusion-python repository as their dependency.

@github-actions github-actions bot added documentation Improvements or additions to documentation development-process Related to development process of DataFusion core Core DataFusion crate common Related to common crate labels Nov 7, 2025
@timsaucer timsaucer force-pushed the chore/remove-pyarrow branch from c8967f5 to 077fb8c Compare November 7, 2025 12:14
@timsaucer timsaucer marked this pull request as ready for review November 7, 2025 12:43
Copy link
Contributor

@milenkovicm milenkovicm left a comment

Choose a reason for hiding this comment

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

Minor comment, would it be appropriate to include this breaking change in the “Upgrade Guide”? You already mentioned in the issue description that “if you need it, use DataFusion Python”.

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.

Thanks @timsaucer -- I agree with @milenkovicm that a note in the upgrade guide would be best

I also suggest we wait until we have made the branch-51 for the 51 release to merge this one so it gets more "bake time" on main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

common Related to common crate core Core DataFusion crate development-process Related to development process of DataFusion documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RFC: Should we remove pyarrow feature from datafusion core

3 participants