Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/22362#discussion_r216107385 --- Diff: R/pkg/R/sparkR.R --- @@ -624,8 +624,8 @@ sparkConfToSubmitOps[["spark.driver.extraClassPath"]] <- "--driver-class-path" sparkConfToSubmitOps[["spark.driver.extraJavaOptions"]] <- "--driver-java-options" sparkConfToSubmitOps[["spark.driver.extraLibraryPath"]] <- "--driver-library-path" sparkConfToSubmitOps[["spark.master"]] <- "--master" -sparkConfToSubmitOps[["spark.yarn.keytab"]] <- "--keytab" --- End diff -- @felixcheung This seems wrong, since existing apps are using the old config, not the new one, so you may need some backwards compatibility code.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org