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

    https://github.com/apache/spark/pull/21278#discussion_r187169097
  
    --- Diff: R/pkg/R/sparkR.R ---
    @@ -163,6 +163,10 @@ sparkR.sparkContext <- function(
         submitOps <- getClientModeSparkSubmitOpts(
             Sys.getenv("SPARKR_SUBMIT_ARGS", "sparkr-shell"),
             sparkEnvirMap)
    +    tryCatch(checkJavaVersion(),
    +             error = function(e) {
    +               stop("Java version check failed")
    --- End diff --
    
    Right- Initially I had thought of returning an error code. But I think stop 
within the function is good enough. Removed this


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to