Skip to content

GH-54489: [CI][Integration] Install jpype1 from conda-forge in conda-integration image - #54490

Merged
kou merged 1 commit into
apache:mainfrom
CurtHagenlocher:ci-integration-conda-jpype1
Oct 19, 2026
Merged

kou merged 1 commit into
apache:mainfrom
CurtHagenlocher:ci-integration-conda-jpype1

Conversation

@CurtHagenlocher

@CurtHagenlocher CurtHagenlocher commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Rationale for this change

The conda-integration image doesn't pin Python, so it now gets Python 3.15. ci/scripts/integration_arrow.sh installs jpype1 with pip, and PyPI has no cp315 wheel for jpype1 yet. pip falls back to building from source, which fails because the image doesn't include Apache Ant. See #54489 for details.

What changes are included in this PR?

Install jpype1 from conda-forge in conda-integration.dockerfile, as conda-python-jpype.dockerfile already does. conda-forge has builds for Python 3.15 on both linux-64 and linux-aarch64.

integration_arrow.sh is unchanged: its pip install jpype1 finds the conda package already installed and does nothing. It still installs jpype1 when the script runs outside this image.

Are these changes tested?

Yes. The Integration job in apache/arrow-dotnet builds this image from scratch and fails with the error above. I ran that job on a fork with the Arrow checkout pointed at this branch, and it got past the jpype1 installation that used to fail. When I opened this PR the run was still in progress: https://github.com/CurtHagenlocher/arrow-dotnet/actions/runs/39150917107

Are there any user-facing changes?

No.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

🤖 Generated with Claude Code

…n image

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 18, 2026 18:21
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #54489 has been automatically assigned in GitHub to PR creator.

@CurtHagenlocher

Copy link
Copy Markdown
Contributor Author

The verification run linked in the description has finished, and the Integration job passed: https://github.com/CurtHagenlocher/arrow-dotnet/actions/runs/39150917107/job/117744207024

  • The image was built with Python 3.15.0, and conda installed jpype1-1.7.1-py315h62b6238_1. pip install jpype1 then reported Requirement already satisfied.
  • The Java C Data Interface tests ran in both directions (.NET → Java and Java → .NET).
  • Result: 0 failures, 337 skips. That skip count matches the last successful run before the Python 3.15 change (Oct 12).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The narrow change follows the existing JPype image pattern, with successful integration validation and no unresolved findings.

No open issues 🎉

What changed in this PR

Preinstalls jpype1 from conda-forge in the integration image, avoiding source-build failures when PyPI lacks wheels for the selected Python version.

Changes:

  • Adds jpype1 to the existing mamba installation.
  • Documents the reason; leaves the integration script unchanged.
File Description
ci/​docker/​conda-integration.dockerfile Installs and documents the conda-provided JPype dependency.

Review effort: Balanced

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@kou
kou merged commit d67d78f into apache:main Oct 19, 2026
59 checks passed
@kou kou removed the awaiting review Awaiting review label Oct 19, 2026
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants