Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10057#discussion_r48175719
--- Diff:
core/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcher.scala
---
@@ -50,7 +50,11 @@ private[mesos] class MesosClusterDispatcher(
extends Logging {
private val publicAddress =
Option(conf.getenv("SPARK_PUBLIC_DNS")).getOrElse(args.host)
- private val recoveryMode = conf.get("spark.mesos.deploy.recoveryMode",
"NONE").toUpperCase()
+ private val recoveryMode =
conf.getOption("spark.mesos.deploy.recoverMode").map { mode =>
+ logWarning("spark.mesos.deploy.recoverMode is deprecated. Please
configure " +
--- End diff --
I don't think it's even worth logging a warning. It wasn't documented so
any user who was using the old config somehow found out about it from the code,
knowing that it's not officially supported. I'd rather keep the code simpler
than add a warning that I doubt will be useful for many.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
