Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/13599#discussion_r164032919 --- Diff: core/src/main/scala/org/apache/spark/deploy/PythonRunner.scala --- @@ -39,12 +39,17 @@ object PythonRunner { val pyFiles = args(1) val otherArgs = args.slice(2, args.length) val sparkConf = new SparkConf() - val pythonExec = sparkConf.get(PYSPARK_DRIVER_PYTHON) + var pythonExec = sparkConf.get(PYSPARK_DRIVER_PYTHON) --- End diff -- Why is this made into a var? You could keep this as a val with a bit of refactoring.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org