Skip to content

Conversation

@jonkeane
Copy link
Member

@jonkeane jonkeane commented Dec 4, 2025

Rationale for this change

Building GCS is fragile in places and we're not sure how many (if anyone) really needs GCS. So let's turn it off by default and get some signal that folks want it back (also hopefully by that time we're on a more modern ABSL)

What changes are included in this PR?

Turn off GCS by default in R builds, also bundle when it is enabled.

Are these changes tested?

Are there any user-facing changes?

No more GCS by default

@nealrichardson
Copy link
Member

@jonkeane Is it still on in the libarrow binaries we build?

@jonkeane
Copy link
Member Author

jonkeane commented Dec 5, 2025

@jonkeane Is it still on in the libarrow binaries we build?

Yes, but I'll run a build to check + confirm https://github.com/jonkeane/arrow/blob/2360a0c82c89581bbc9bc669544b24c0c57ea886/dev/tasks/r/github.packages.yml#L86

@jonkeane
Copy link
Member Author

jonkeane commented Dec 5, 2025

@github-actions crossbow submit r-binary-packages

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

Revision: 2360a0c

Submitted crossbow builds: ursacomputing/crossbow @ actions-85181bb671

Task Status
r-binary-packages GitHub Actions

@jonkeane
Copy link
Member Author

jonkeane commented Dec 5, 2025

Copy link
Member

@nealrichardson nealrichardson left a comment

Choose a reason for hiding this comment

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

Are there any docs we need to update about what is on by default (on CRAN or otherwise)?

@jonkeane
Copy link
Member Author

jonkeane commented Dec 8, 2025

Are there any docs we need to update about what is on by default (on CRAN or otherwise)?

We have an ambiguous statement that it might not be enabled at https://arrow.apache.org/docs/r/articles/fs.html#s3-and-gcs-support-on-linux which is all still accurate (and honestly was slightly less accurate before this).

I'm going to create a new issue as a follow on because I think we want to look at what the right story for "install arrow with all the bells and whistles" is.

@jonkeane jonkeane merged commit 7d11484 into apache:main Dec 8, 2025
10 checks passed
@jonkeane jonkeane deleted the GCS_off_by_default branch December 8, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants