tgravescs commented on a change in pull request #28874:
URL: https://github.com/apache/spark/pull/28874#discussion_r447006726
##########
File path:
core/src/main/scala/org/apache/spark/deploy/rest/RestSubmissionClient.scala
##########
@@ -411,7 +411,7 @@ private[spark] object RestSubmissionClient {
// SPARK_HOME and SPARK_CONF_DIR are filtered out because they are usually
wrong
// on the remote machine (SPARK-12345) (SPARK-25934)
- private val BLACKLISTED_SPARK_ENV_VARS = Set("SPARK_ENV_LOADED",
"SPARK_HOME", "SPARK_CONF_DIR")
+ private val EXCLUDED_SPARK_ENV_VARS = Set("SPARK_ENV_LOADED", "SPARK_HOME",
"SPARK_CONF_DIR")
Review comment:
yes, the idea was to change it to be something specific for how it was
used. If you have other ideas please let us know.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
