Skip to content

[SPARK-32491][INFRA] Do not install SparkR in test-only mode in testing script #29300

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.

Sign up for GitHub

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

Closed
HyukjinKwon wants to merge 1 commit into from
Closed

[SPARK-32491][INFRA] Do not install SparkR in test-only mode in testing script #29300

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Do not install SparkR in test-only mode in testing script
  • Loading branch information
@HyukjinKwon
HyukjinKwon committed Jul 30, 2020
commit b831229bd25b9d4e127c37a65bb7040baedc41c3
14 changes: 8 additions & 6 deletions dev/run-tests.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -609,11 +609,14 @@ def main():
" install one and retry.")
sys.exit(2)

# install SparkR
if which("R"):
run_cmd([os.path.join(SPARK_HOME, "R", "install-dev.sh")])
else:
print("Cannot install SparkR as R was not found in PATH")
# Install SparkR
should_only_test_modules = opts.modules is not None
if not should_only_test_modules:
# If tests modules are specified, we will not run R linter.
if which("R"):
run_cmd([os.path.join(SPARK_HOME, "R", "install-dev.sh")])
else:
print("Cannot install SparkR as R was not found in PATH")

if os.environ.get("AMPLAB_JENKINS"):
# if we're on the Amplab Jenkins build servers setup variables
Expand Down Expand Up @@ -643,7 +646,6 @@ def main():
changed_modules = None
test_modules = None
changed_files = None
should_only_test_modules = opts.modules is not None
included_tags = []
excluded_tags = []
if should_only_test_modules:
Expand Down