Skip to content

cythonize from setup.py to support python 3.11 #32

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 1 commit into from
Mar 7, 2025

Conversation

cbouss
Copy link
Contributor

@cbouss cbouss commented Dec 13, 2022

Similar to #31 but running cython at build time.

@ecederstrand
Copy link

Also works for Python 3.12.

@ecederstrand
Copy link

Trying to get attention from someone at Cloudera to maintain this package. @ihodes maybe?

@ecederstrand
Copy link

Or possibly @phunt or @lajosrodek?

@csringhofer
Copy link
Collaborator

@ecederstrand
Thanks for working on this!
Checked out the PR and could create packages with different Python3 versions, including 3.11 and 3.12.
Couldn't compile it with Python 2.7 though, getting an error from pip_self_version_check.
Did you test it with 2.7?

Python 2.7 support should be dropped soon, but it would be nice to have a final release for it.

@ecederstrand
Copy link

ecederstrand commented May 29, 2024

Python 2.7 was EOL almost four years ago. I haven't tested it and personally don't see any value maintaining support for it.

@csringhofer
Copy link
Collaborator

impala-shell still supports Python 2.7. My plan was to test this change via impala-shell, which turned out to be tricky due the build system being tied to Python2
Nevertheless, in the end it worked and could connect with impala-shell using Kerberos on Python3.11/3.12

Thanks again for working on this!

@csringhofer csringhofer merged commit b8ce84c into cloudera:master Mar 7, 2025
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.

3 participants