Skip to content
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

Exclude Python 3.9.7 due to import error in catalog module #526

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

ndrluis
Copy link
Collaborator

@ndrluis ndrluis commented Mar 15, 2024

Adjusts the supported Python versions to exclude 3.9.7, as importing the catalog module on this version results in an unexpected error. The issue does not occur in versions 3.9.6 or 3.9.8, indicating a specific incompatibility with Python 3.9.7.

Related to #500, #497

@ndrluis
Copy link
Collaborator Author

ndrluis commented Mar 16, 2024

Related python issue: https://bugs.python.org/issue45081

@sungwy
Copy link
Collaborator

sungwy commented Jul 31, 2024

This sounds like a great idea @ndrluis - thank you for identifying this issue and putting in the fix. Could we ask for you to rebase your PR against the current main, and run poetry lock with your proposed python version change?

@ndrluis
Copy link
Collaborator Author

ndrluis commented Jul 31, 2024

@sungwy Done!

Adjusts the supported Python versions to exclude 3.9.7, as importing the
catalog module on this version results in an unexpected error.
The issue does not occur in versions 3.9.6 or 3.9.8, indicating a specific
incompatibility with Python 3.9.7.
@kevinjqliu
Copy link
Contributor

With this change, what do I see as a user with python 3.9.7? Does it show that 3.9.7 is not supported and I should use a supported version?

@ndrluis
Copy link
Collaborator Author

ndrluis commented Aug 6, 2024

Yes @kevinjqliu!

This is a example
image

pyproject.toml Outdated Show resolved Hide resolved
@ndrluis ndrluis requested a review from kevinjqliu August 8, 2024 01:41
Co-authored-by: Kevin Liu <[email protected]>
Copy link
Contributor

@kevinjqliu kevinjqliu left a comment

Choose a reason for hiding this comment

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

LGTM!
I've tested with python 3.9.7 and running make install
Here's the error log

The currently activated Python version 3.9.7 is not supported by the project (>=3.8, <4.0, !=3.9.7).
Trying to find and use a compatible version.

@kevinjqliu kevinjqliu merged commit 85df8d4 into apache:main Aug 8, 2024
7 checks passed
@Fokko Fokko added this to the PyIceberg 0.7.1 release milestone Aug 8, 2024
sungwy pushed a commit that referenced this pull request Aug 9, 2024
* Exclude Python 3.9.7 due to import error in catalog module

Adjusts the supported Python versions to exclude 3.9.7, as importing the
catalog module on this version results in an unexpected error.
The issue does not occur in versions 3.9.6 or 3.9.8, indicating a specific
incompatibility with Python 3.9.7.

* Update pyproject.toml

Co-authored-by: Kevin Liu <[email protected]>

---------

Co-authored-by: Kevin Liu <[email protected]>
sungwy pushed a commit that referenced this pull request Aug 9, 2024
* Exclude Python 3.9.7 due to import error in catalog module

Adjusts the supported Python versions to exclude 3.9.7, as importing the
catalog module on this version results in an unexpected error.
The issue does not occur in versions 3.9.6 or 3.9.8, indicating a specific
incompatibility with Python 3.9.7.

* Update pyproject.toml

Co-authored-by: Kevin Liu <[email protected]>

---------

Co-authored-by: Kevin Liu <[email protected]>
@ndrluis ndrluis deleted the version-support branch August 20, 2024 18:13
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.

4 participants