Skip to content

GH-45787: [Integration][CI] Fix Rust integration build, workaround dep - #45801

Closed
alamb wants to merge 1 commit into
apache:mainfrom
alamb:alamb/fix_rust_integration
Closed

GH-45787: [Integration][CI] Fix Rust integration build, workaround dep#45801
alamb wants to merge 1 commit into
apache:mainfrom
alamb:alamb/fix_rust_integration

Conversation

@alamb

@alambalamb commented Mar 14, 2025

Copy link
Copy Markdown
Contributor

Rationale for this change

Our integration jobs are failing due to an old version of rustc being used with a new rust dependency

NOTE this is an alternate workaround to this PR proposed by @raulcd

I think the approach of updating the version of rustc in the integration image is the right long term solution, but I think this PR would be enough to get the tests running successfully again

What changes are included in this PR?

Pin the dependency that causes compilation to fail to the previous version

Are these changes tested?

The currently failing CI job is now successful

Are there any user-facing changes?

This image might be used on other repositories (nanoarrow, arrow-rust, arrow-go, arrow-java, ...). It might also affect those?

@github-actions

Copy link
Copy Markdown

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:

@github-actionsgithub-actionsBot added the awaiting committer review Awaiting committer review label Mar 14, 2025
@alamb
alamb marked this pull request as draft March 14, 2025 21:17
@koukou changed the title [GH-45787]: [Integration] Fix Rust integration build, workaround depGH-45787: [Integration] Fix Rust integration build, workaround depMar 15, 2025
@koukou changed the title GH-45787: [Integration] Fix Rust integration build, workaround depGH-45787: [Integration][CI] Fix Rust integration build, workaround depMar 15, 2025
@alamb

Copy link
Copy Markdown
ContributorAuthor

Let's go with #45790

@alambalamb closed this Mar 15, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting committer reviewAwaiting committer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@alamb