Skip to content

Commit

Permalink
GH-44389: [Java][Integration][Release] Use Python 3.12 for verify-rc-…
Browse files Browse the repository at this point in the history
…source-integration-linux-conda-latest-amd64 (#44395)

### Rationale for this change

Because JPype doesn't support Python 3.13 yet.

See also:
* GH-44389
* https://discuss.python.org/t/api-for-python-3-13-prevents-use-of-3rd-party-gc-allocators/62709/5

### What changes are included in this PR?

Pin Python 3.12.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #44389

Lead-authored-by: Raúl Cumplido <[email protected]>
Co-authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
  • Loading branch information
kou authored Oct 16, 2024
1 parent 0577ff6 commit d0e7d07
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 9 additions & 0 deletions dev/tasks/verify-rc/github.linux.amd64.docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,15 @@ jobs:
{% if distro == 'almalinux' and target|upper == 'PYTHON' %}
-e ARROW_GANDIVA=OFF \
{% endif %}
{% if distro == "conda" and target == "integration" %}
{# JPype doesn't work with Python 3.13.
# See also:
# * https://discuss.python.org/t/api-for-python-3-13-prevents-use-of-3rd-party-gc-allocators/62709/5
# * GH-44386
# * GH-44389
#}
-e PYTHON_VERSION="3.12" \
{% endif %}
-e VERIFY_RC="{{ rc|default("") }}" \
-e TEST_DEFAULT=0 \
-e TEST_{{ target|upper }}=1 \
Expand Down
2 changes: 1 addition & 1 deletion dev/tasks/verify-rc/github.macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ jobs:
TEST_DEFAULT: 0
TEST_{{ target|upper }}: 1
{% if use_conda %}
USE_CONDA: 1
{% if target == "integration" %}
# JPype doesn't work with Python 3.13.
# See also:
Expand All @@ -74,6 +73,7 @@ jobs:
# * GH-44389
PYTHON_VERSION: "3.12"
{% endif %}
USE_CONDA: 1
{% endif %}
run: |
arrow/dev/release/verify-release-candidate.sh {{ release|default("") }} {{ rc|default("") }}

0 comments on commit d0e7d07

Please sign in to comment.