Github user zjffdu commented on a diff in the pull request: https://github.com/apache/spark/pull/13599#discussion_r164069473 --- 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 -- I am afraid I have to use var, because `VirtualenvFactory` also need `pythonExec` as constructor arg.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org