HyukjinKwon commented on code in PR #40954:
URL: https://github.com/apache/spark/pull/40954#discussion_r1184620325


##########
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala:
##########
@@ -160,8 +164,20 @@ private[spark] abstract class BasePythonRunner[IN, OUT](
       envVars.put("PYTHON_FAULTHANDLER_DIR", 
BasePythonRunner.faultHandlerLogDir.toString)
     }
 
+    // If local property "pythonEnv.allSparkNodesReadableRootEnvDir" is set,
+    // then driver side has initiated a python environment in this directory.
+    // `env.createPythonWorker` can reuse the python environment set up in 
driver side.
+    val rootPythonEvnDir = Option(
+      context.getLocalProperty("pythonEnv.allSparkNodesReadableRootEnvDir")

Review Comment:
   Can we start the name with `spark`.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to