-
Notifications
You must be signed in to change notification settings - Fork 4k
MINOR: [R][CI] Use docker.io/library/r-base instead of docker.io/rocker/r-base
#14607
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: SHIMA Tatsuya <[email protected]>
|
@assignUser @nealrichardson you might want to take a look on this one |
|
It is maintained by the rocker community so they are probably updated pretty much at the same time. But the library/r-base image is a multich arch image which is actually a reason to switch. We would not be taking advantage of it immediatly but it should make it easier to test R stuff on different architectures. |
|
@github-actions crossbow submit test-r-library-* test-r-minimal |
|
Revision: 0bbdc3d Submitted crossbow builds: ursacomputing/crossbow @ actions-11872cebf2
|
|
@assignUser Will this PR fix the problem alluded to here or do we still need to do something else? |
|
This is not related to #14654 (comment) because this doesn't change |
|
@assignUser @kou Can you merge this? This is a really minor change. |
kou
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
|
Thanks! |
|
Benchmark runs are scheduled for baseline = 069a45f and contender = 9b0f3d7. 9b0f3d7 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
|
['Python', 'R'] benchmarks have high level of regressions. |

Unless there is a special reason, it is better to use
docker.io/library/r-base, Docker Official Images.It is built from the same source, but
docker.io/library/r-baseis always up-to-date because it is built automatically.