Skip to content

Commit 3efdf35

Browse files
shaneknappdbtsai
authored andcommitted
[SPARK-24908][R][STYLE] removing spaces to make lintr happy
## What changes were proposed in this pull request? during my travails in porting spark builds to run on our centos worker, i managed to recreate (as best i could) the centos environment on our new ubuntu-testing machine. while running my initial builds, lintr was crashing on some extraneous spaces in test_basic.R (see: https://amplab.cs.berkeley.edu/jenkins/job/spark-master-test-sbt-hadoop-2.6-ubuntu-test/862/console) after removing those spaces, the ubuntu build happily passed the lintr tests. ## How was this patch tested? i then tested this against a modified spark-master-test-sbt-hadoop-2.6 build (see https://amplab.cs.berkeley.edu/jenkins/view/RISELab%20Infra/job/testing-spark-master-test-with-updated-R-crap/4/), which scp'ed a copy of test_basic.R in to the repo after the git clone. everything seems to be working happily. Author: shane knapp <[email protected]> Closes apache#21864 from shaneknapp/fixing-R-lint-spacing.
1 parent fc21f19 commit 3efdf35

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

R/pkg/inst/tests/testthat/test_basic.R

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@
1818
context("basic tests for CRAN")
1919

2020
test_that("create DataFrame from list or data.frame", {
21-
tryCatch( checkJavaVersion(),
21+
tryCatch(checkJavaVersion(),
2222
error = function(e) { skip("error on Java check") },
23-
warning = function(e) { skip("warning on Java check") } )
23+
warning = function(e) { skip("warning on Java check") })
2424

2525
sparkR.session(master = sparkRTestMaster, enableHiveSupport = FALSE,
2626
sparkConfig = sparkRTestConfig)
@@ -54,9 +54,9 @@ test_that("create DataFrame from list or data.frame", {
5454
})
5555

5656
test_that("spark.glm and predict", {
57-
tryCatch( checkJavaVersion(),
57+
tryCatch(checkJavaVersion(),
5858
error = function(e) { skip("error on Java check") },
59-
warning = function(e) { skip("warning on Java check") } )
59+
warning = function(e) { skip("warning on Java check") })
6060

6161
sparkR.session(master = sparkRTestMaster, enableHiveSupport = FALSE,
6262
sparkConfig = sparkRTestConfig)

0 commit comments

Comments
 (0)