-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
ARROW-14217: [Python][CI] Add support for python 3.10 #11316
Conversation
@github-actions crossbow submit cp310 |
Revision: cd3da84 Submitted crossbow builds: ursacomputing/crossbow @ actions-883 |
@github-actions crossbow submit wheel-macos-big-sur-cp310* |
Revision: 583301f Submitted crossbow builds: ursacomputing/crossbow @ actions-884
|
@github-actions crossbow submit cp310 |
Revision: 583301f Submitted crossbow builds: ursacomputing/crossbow @ actions-885 |
@github-actions crossbow submit wheel-manylinuxcp310 |
Revision: af016f3 Submitted crossbow builds: ursacomputing/crossbow @ actions-886
|
@github-actions crossbow submit wheel-manylinuxcp310 |
Revision: 93fc1b7 Submitted crossbow builds: ursacomputing/crossbow @ actions-887
|
Why don't you start by adding a GHA entry for Python 3.10? |
You mean an additional conda based build? |
@github-actions crossbow submit wheel-manylinuxcp310 |
Hmm, my bad, there's no conda package available. |
@github-actions crossbow submit wheel-manylinuxcp310 |
Revision: ee6e3c6 Submitted crossbow builds: ursacomputing/crossbow @ actions-891
|
The manylinux builds are working, but the testing docker images have not been released yet - matter of days I guess. For the rest of the wheels we need to wait for the next numpy release. |
@github-actions crossbow submit wheel-manylinuxcp310 The |
Revision: ee6e3c6 Submitted crossbow builds: ursacomputing/crossbow @ actions-896
|
Revision: ee6e3c6 Submitted crossbow builds: ursacomputing/crossbow @ actions-897
|
Revision: de75bb5 Submitted crossbow builds: ursacomputing/crossbow @ actions-939 |
Revision: de75bb5 Submitted crossbow builds: ursacomputing/crossbow @ actions-940 |
Thanks @pitrou for the fix! |
@github-actions crossbow submit test-conda-python-3.10 |
Revision: 90a0fd3 Submitted crossbow builds: ursacomputing/crossbow @ actions-941
|
"test-conda-python-3.10" will fail as long as Numpy and other packages are not available yet for Python 3.10 on conda-forge. |
@kszucs It seems vcpkg is building both debug and release versions of dependencies? See example
|
For some reason, it looks like the 3.10 build doesn't use the Numpy binary wheels: |
Strange, the selected triplet file should restrict it to release builds. |
Seems like that's not true for all of the packages. |
We may try to restrict that like in the macos builds. |
We may also need to bump the manylinux image tag because the interpreter path indicates cypthon version 3.10-rc2 |
Ahh, numpy only has wheels for |
The wheel has the right tag apparently: "pyarrow-6.0.0.dev470-cp310-cp310-manylinux_2_12_x86_64.manylinux2010_x86_64.whl". |
I think it's worth waiting for:
|
I'm not sure what waiting would bring here. We have a working fine to fix the tests already, and we can produce manylinux wheels. We can add the rest later. |
@github-actions crossbow submit wheel-manylinux* |
@@ -47,7 +47,7 @@ jobs: | |||
{{ macros.github_upload_releases("arrow/python/repaired_wheels/*.whl")|indent }} | |||
{{ macros.github_upload_gemfury("arrow/python/repaired_wheels/*.whl")|indent }} | |||
|
|||
{% if arrow.branch == 'master' %} | |||
{% if True or arrow.branch == 'master' %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to remove before merge.
Revision: 626987a Submitted crossbow builds: ursacomputing/crossbow @ actions-954 |
We can merge it now as well, but then we are going to have failing builds for the other platforms until numpy gains the new wheels. Ideally the only thing we'll need to do is to build and push the windows image for python 3.10, otherwise the builds should fix themselves. Does that sound good to you? |
Yes, that sounds good to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, merging on green
Benchmark runs are scheduled for baseline = bea1701 and contender = 15f4e56. 15f4e56 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
No description provided.