Skip to content

Commit

Permalink
apacheGH-43045: [CI][Python] Pin openjdk=17 in python substrait integ…
Browse files Browse the repository at this point in the history
…ration (apache#43051)

### Rationale for this change

Substrait builds and releases using JDK 17.

In the Substrait repo, Gradle was updated from 7.4.2 -> 8.8. substrait-io/substrait-java@77e79ad

Gradle 8.8 can not be used with openjdk 22 yet, which is the latest version downloaded from condaforge. My testing showed that openjdk 21 also fails with the same error.

### What changes are included in this PR?

* Pin openjdk=17

### Are these changes tested?

Testing via crossbow

### Are there any user-facing changes?

No
* GitHub Issue: apache#43045

Authored-by: Dane Pitkin <dpitkin@apache.org>
Signed-off-by: Dane Pitkin <dpitkin@apache.org>
  • Loading branch information
danepitkin committed Jun 26, 2024
1 parent 1815a67 commit 68db662
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion ci/docker/conda-python-substrait.dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,19 @@ FROM ${repo}:${arch}-conda-python-${python}
COPY ci/conda_env_python.txt \
ci/conda_env_sphinx.txt \
/arrow/ci/

# Note: openjdk is pinned to 17 because the
# substrait repo currently pins to jdk 17.
# Newer jdk versions are currently failing
# due to the recent upgrade to Gradle 8 via
# install_substrait_consumer.sh.
# https://github.com/substrait-io/substrait-java/issues/274
RUN mamba install -q -y \
--file arrow/ci/conda_env_python.txt \
--file arrow/ci/conda_env_sphinx.txt \
$([ "$python" == "3.9" ] && echo "pickle5") \
python=${python} openjdk \
python=${python} \
openjdk=17 \
nomkl && \
mamba clean --all

Expand Down

0 comments on commit 68db662

Please sign in to comment.