This is an automated email from the ASF dual-hosted git repository.
srowen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 4f99e91f352 [SPARK-39244][INFRA] Use `--no-echo` instead of `--slave`
in R 4.0+
4f99e91f352 is described below
commit 4f99e91f352f070029a38bdb5247057cb648ddb3
Author: William Hyun <[email protected]>
AuthorDate: Sat May 21 09:39:22 2022 -0500
[SPARK-39244][INFRA] Use `--no-echo` instead of `--slave` in R 4.0+
### What changes were proposed in this pull request?
This PR aims to use `--no-echo` instead of `--slave` in R 4.0+
### Why are the changes needed?
- https://github.com/rstudio/rstudio/issues/5923
-
https://github.com/wch/r-source/commit/f1ff49e74593341c74c20de9517f31a22c8bcb04
The R community has already moved to `--no-echo`
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
N/A, this is infra code.
Closes #36622 from williamhyun/slave.
Authored-by: William Hyun <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
---
.../roles/jenkins-worker/tasks/install_spark_build_packages.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/dev/ansible-for-test-node/roles/jenkins-worker/tasks/install_spark_build_packages.yml
b/dev/ansible-for-test-node/roles/jenkins-worker/tasks/install_spark_build_packages.yml
index 663954fde74..413c1dcfb84 100644
---
a/dev/ansible-for-test-node/roles/jenkins-worker/tasks/install_spark_build_packages.yml
+++
b/dev/ansible-for-test-node/roles/jenkins-worker/tasks/install_spark_build_packages.yml
@@ -158,7 +158,7 @@
- r-mathlib
- name: install required R packages via Rscript (default version)
- command: /usr/bin/Rscript --slave --no-save --no-restore-history -e "if (!
('{{ item }}' %in% installed.packages()[,'Package'])) {
install.packages(pkgs='{{ item }}'); print('Added'); } else { print('Already
installed'); }"
+ command: /usr/bin/Rscript --no-echo --no-save --no-restore-history -e "if (!
('{{ item }}' %in% installed.packages()[,'Package'])) {
install.packages(pkgs='{{ item }}'); print('Added'); } else { print('Already
installed'); }"
register: r_result
failed_when: "r_result.rc != 0 or 'had non-zero exit status' in
r_result.stderr"
changed_when: "'Added' in r_result.stdout"
@@ -179,5 +179,5 @@
register: r_check
- name: install lintr v2.0.0
- command: /usr/bin/Rscript --slave --no-save --no-restore-history -e
"devtools::install_github('jimhester/[email protected]')"
+ command: /usr/bin/Rscript --no-echo --no-save --no-restore-history -e
"devtools::install_github('jimhester/[email protected]')"
when: "'lintr' not in r_check.stdout"
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
