Skip to content

GH-47106: [R] Update R package to use R 4.1+ native forward pipe syntax - #47622

Merged
amoeba merged 5 commits into
apache:mainfrom
thisisnic:GH_47106-native_pipe
Oct 6, 2025
Merged

GH-47106: [R] Update R package to use R 4.1+ native forward pipe syntax#47622
amoeba merged 5 commits into
apache:mainfrom
thisisnic:GH_47106-native_pipe

Conversation

@thisisnic

@thisisnicthisisnic commented Sep 22, 2025

Copy link
Copy Markdown
Member

Rationale for this change

Don't need base pipe

What changes are included in this PR?

Update package to use native pipe

Are these changes tested?

Sure

Are there any user-facing changes?

Nah

@github-actions

Copy link
Copy Markdown

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

@thisisnic
thisisnic marked this pull request as ready for review September 24, 2025 06:16
@amoeba

Copy link
Copy Markdown
Member

@github-actions crossbow submit -g r

@github-actions

Copy link
Copy Markdown

Revision: 6892a21

Submitted crossbow builds: ursacomputing/crossbow @ actions-0c305d6d3e

TaskStatus
r-binary-packagesGitHub Actions
r-recheck-mostGitHub Actions
test-r-arrow-backwards-compatibilityGitHub Actions
test-r-depsource-bundledAzure
test-r-depsource-systemGitHub Actions
test-r-dev-duckdbGitHub Actions
test-r-devdocsGitHub Actions
test-r-extra-packagesGitHub Actions
test-r-gcc-11GitHub Actions
test-r-gcc-12GitHub Actions
test-r-install-localGitHub Actions
test-r-install-local-minsizerelGitHub Actions
test-r-linux-as-cranGitHub Actions
test-r-linux-rchkGitHub Actions
test-r-linux-sanitizersGitHub Actions
test-r-linux-valgrindGitHub Actions
test-r-m1-sanGitHub Actions
test-r-macos-as-cranGitHub Actions
test-r-minimal-buildAzure
test-r-offline-maximalGitHub Actions
test-r-offline-minimalAzure
test-r-rhub-debian-gcc-devel-lto-latestAzure
test-r-rhub-debian-gcc-release-custom-ccacheAzure
test-r-rhub-ubuntu-release-latestAzure
test-r-rocker-r-ver-latestAzure
test-r-rstudio-r-base-4.1-focalAzure
test-r-rstudio-r-base-4.2-focalAzure
test-r-ubuntu-22.04GitHub Actions
test-r-versionsGitHub Actions

@amoebaamoeba 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.

This change feels pretty safe but I thought I'd kick off crossbow jobs just in case. Thanks for taking this on @thisisnic.

@amoeba

Copy link
Copy Markdown
Member

Looks like nightly libarrow for R isn't being built and we may need to add an override in the script for test-r-rhub-debian-gcc-release-custom-ccache to run correctly until rhub/debian-gcc-release is updated.

@amoeba
amoeba merged commit 48a2e88 into apache:mainOct 6, 2025
11 checks passed
@amoebaamoeba removed the awaiting committer review Awaiting committer review label Oct 6, 2025
@conbench-apache-arrow

Copy link
Copy Markdown

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

There were no benchmark performance regressions. 🎉

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

zanmato1984 pushed a commit to zanmato1984/arrow that referenced this pull request Oct 15, 2025
…e syntax (apache#47622)
### Rationale for this change
Don't need base pipe
### What changes are included in this PR?
Update package to use native pipe
### Are these changes tested?
Sure
### Are there any user-facing changes?
Nah
* GitHub Issue: apache#47106
Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Bryce Mecum <petridish@gmail.com>
Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
…e syntax (apache#47622)
### Rationale for this change
Don't need base pipe
### What changes are included in this PR?
Update package to use native pipe
### Are these changes tested?
Sure
### Are there any user-facing changes?
Nah
* GitHub Issue: apache#47106
Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Bryce Mecum <petridish@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@thisisnic@amoeba