Skip to content

Upgrade pyarrow minimum version to 7.0 #51151

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.

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

Merged
merged 8 commits into from
Feb 7, 2023
Merged

Conversation

phofl
Copy link
Member

@phofl phofl commented Feb 3, 2023

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

@phofl phofl requested a review from mroeschke February 3, 2023 19:51
@phofl phofl changed the title Upgrade ci minimum version to 7.0 Upgrade pyarrow minimum version to 7.0 Feb 3, 2023
@jbrockmendel
Copy link
Member

Was this discussed somewhere? I'm fine with bumping this pretty aggressively but not sure if we have a policy

@phofl
Copy link
Member Author

phofl commented Feb 4, 2023

Informal policy was support for packages released up to a year ago for 2.0, pyarrow 7 was released 2nd or 3rd of February 2022

@jreback
Copy link
Contributor

jreback commented Feb 4, 2023

+1 here for bumping

@mroeschke mroeschke added Dependencies Required and optional dependencies Arrow pyarrow functionality labels Feb 6, 2023
@mroeschke mroeschke added this to the 2.0 milestone Feb 7, 2023
@mroeschke mroeschke merged commit d8a1a0d into pandas-dev:main Feb 7, 2023
@mroeschke
Copy link
Member

Thanks @phofl

@phofl phofl deleted the pyarrow branch February 7, 2023 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arrow pyarrow functionality Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants