Skip to content

GH-49459: [R][CI] Use RHEL-9 binaries on Amazon Linux 2023 builds#49460

Merged
raulcd merged 1 commit intoapache:mainfrom
thisisnic:GH-49459-amazon-linux-binaries
Mar 6, 2026
Merged

GH-49459: [R][CI] Use RHEL-9 binaries on Amazon Linux 2023 builds#49460
raulcd merged 1 commit intoapache:mainfrom
thisisnic:GH-49459-amazon-linux-binaries

Conversation

@thisisnic
Copy link
Member

@thisisnic thisisnic commented Mar 5, 2026

Rationale for this change

It takes 45 mins to install the duckdb R package in the benchmarks as it builds from source

What changes are included in this PR?

Use the RSPM RHEL9 binaries instead by mapping Amazon Linux 2023 to RHEL9

Are these changes tested?

I did locally but I'll ask Ursabot to benchmark or whatever

Are there any user-facing changes?

Nah

@thisisnic
Copy link
Member Author

@ursabot please benchmark

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Mar 5, 2026
Copy link
Member

@rok rok left a comment

Choose a reason for hiding this comment

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

LGTM!

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Mar 5, 2026
Copy link
Member

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

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

I'm fine with this, though we probably could clean up all this code rather than adding to it by using Posit Public Package Manager's many linux packages

https://posit.co/blog/introducing-portable-linux-r-binary-packages/

@thisisnic
Copy link
Member Author

I'm fine with this, though we probably could clean up all this code rather than adding to it by using Posit Public Package Manager's many linux packages

https://posit.co/blog/introducing-portable-linux-r-binary-packages/

I opened #49461 to follow up on this

@raulcd raulcd merged commit 2ce4e66 into apache:main Mar 6, 2026
26 of 27 checks passed
@raulcd raulcd removed the awaiting merge Awaiting merge label Mar 6, 2026
@github-actions github-actions bot added the awaiting merge Awaiting merge label Mar 6, 2026
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 2ce4e66.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

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.

4 participants