Skip to content

Conversation

@kou
Copy link
Member

@kou kou commented Sep 30, 2025

Rationale for this change

Our latest Go version detection is broken. It doesn't detect the latest version.

It caused Apache Arrow Go install failure.

What changes are included in this PR?

Use conda to install Go.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

Our latest Go version detection is broken. It doesn't detect the
latest version.
@kou kou changed the title [CI][Integration] Install Go by conda [CI][Integration] Fix Go build failure Sep 30, 2025
@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Sep 30, 2025
@github-actions
Copy link

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

@kou kou changed the title [CI][Integration] Fix Go build failure GH-47673: [CI][Integration] Fix Go build failure Sep 30, 2025
@github-actions
Copy link

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

@kou kou merged commit 935f318 into apache:main Sep 30, 2025
36 of 38 checks passed
@kou kou deleted the ci-integration-go branch September 30, 2025 21:58
@kou kou removed the awaiting committer review Awaiting committer review label Sep 30, 2025
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 935f318.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.

@pitrou
Copy link
Member

pitrou commented Oct 1, 2025

Thanks @kou !

zanmato1984 pushed a commit to zanmato1984/arrow that referenced this pull request Oct 15, 2025
### Rationale for this change

Our latest Go version detection is broken. It doesn't detect the latest version.

It caused Apache Arrow Go install failure.

### What changes are included in this PR?

Use conda to install Go.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: apache#47673

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
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.

2 participants