thisisnic commented on PR #49038: URL: https://github.com/apache/arrow/pull/49038#issuecomment-3808630505
This PR adds an `install_r()` function that installs R via [https://github.com/r-lib/rig](rig) instead of using conda R. The goal is to speed up R benchmarks by allowing RSPM binary packages (like duckdb) to be used instead of compiling from source. However, this PR alone isn't sufficient - the benchmark CI infrastructure lives in a separate repo (https://github.com/arctosalliance/arrow-benchmarks-ci) which currently includes r in its conda environment creation. Even though this PR adds `install_r()`, conda R gets installed first, so the `command -v R` check passes and rig is never used. Companion PR needed: arctosalliance/arrow-benchmarks-ci#1 - removes r from conda env and calls the `install_r` hook from this PR. The arctosalliance PR depends on this one being merged first. -- 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]
