Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/15666#discussion_r143015274
  
    --- Diff: R/pkg/tests/fulltests/test_context.R ---
    @@ -167,6 +167,18 @@ test_that("spark.lapply should perform simple 
transforms", {
       sparkR.session.stop()
     })
     
    +test_that("add jar should work and allow usage of the jar on the driver 
node", {
    +  sparkR.sparkContext()
    +
    +  destDir <- file.path(tempdir(), "testjar")
    +  jarName <- callJStatic("org.apache.spark.TestUtils", "createDummyJar",
    +      destDir, "sparkrTests", "DummyClassForAddJarTest")
    --- End diff --
    
    little nit:
    
    ```r
    jarName <- callJStatic("org.apache.spark.TestUtils", "createDummyJar",
                            destDir, "sparkrTests", "DummyClassForAddJarTest")
    ```



---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to