Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19631#discussion_r150916079
--- Diff: core/src/main/scala/org/apache/spark/SecurityManager.scala ---
@@ -551,13 +553,10 @@ private[spark] class SecurityManager(
private[spark] object SecurityManager {
- val SPARK_AUTH_CONF: String = "spark.authenticate"
- val SPARK_AUTH_SECRET_CONF: String = "spark.authenticate.secret"
+ val SPARK_AUTH_CONF = NETWORK_AUTH_ENABLED.key
+ val SPARK_AUTH_SECRET_CONF = "spark.authenticate.secret"
--- End diff --
In a separate change maybe. There are too many references to this right
now, it would be really noisy here.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
