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

    https://github.com/apache/spark/pull/19840#discussion_r154028953
  
    --- Diff: 
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
    @@ -57,9 +57,14 @@ private[spark] abstract class BasePythonRunner[IN, OUT](
     
       require(funcs.length == argOffsets.length, "argOffsets should have the 
same length as funcs")
     
    +  private val env = SparkEnv.get
    +  private val conf = env.conf
    +
       // All the Python functions should have the same exec, version and 
envvars.
       protected val envVars = funcs.head.funcs.head.envVars
    -  protected val pythonExec = funcs.head.funcs.head.pythonExec
    +  protected val pythonExec = 
conf.getOption("spark.executorEnv.PYSPARK_DRIVER_PYTHON")
    --- End diff --
    
    Shouldn't this be handled in `deploy/PythonRunner`?


---

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

Reply via email to