nealrichardson commented on code in PR #35523:
URL: https://github.com/apache/arrow/pull/35523#discussion_r1355575292
##########
.github/workflows/r.yml:
##########
@@ -234,8 +234,8 @@ jobs:
fail-fast: false
matrix:
config:
- - { rtools: 42, rversion: "4.2" }
Review Comment:
Valid concern, though in regular CI we only build the ucrt64 version, the
32-bit is only nightly, so the build time is less of an issue there. (I think
the libarrow builds are also parallelized.)
That said, we already "violate" the tidyverse standard for version
compatibility (Windows only) because we require R 4.0 on Windows due to C++17.
I think I'm landing in the same place as @paleolimbot here: leave it in
unless/until the 32-bit builds are causing undue developer burden, which it
seems like today they are not.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
