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

    https://github.com/apache/spark/pull/19589#discussion_r147545247
  
    --- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
    @@ -36,6 +36,12 @@ opts_hooks$set(eval = function(options) {
       }
       options
     })
    +r_tmp_dir <- tempdir()
    +tmp_arg <- paste("-Djava.io.tmpdir=", r_tmp_dir, sep = "")
    +sparkSessionConfig <- list(spark.driver.extraJavaOptions = tmp_arg,
    +                           spark.executor.extraJavaOptions = tmp_arg)
    +old_java_opt <- Sys.getenv("_JAVA_OPTIONS")
    +new_java_opt <- Sys.setenv("_JAVA_OPTIONS"=paste("-XX:-UsePerfData", 
old_java_opt, sep = " "))
    --- End diff --
    
    don't need `new_java_opt`?


---

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

Reply via email to