Github user ArtRand commented on a diff in the pull request: https://github.com/apache/spark/pull/20167#discussion_r161385080 --- Diff: resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala --- @@ -80,10 +80,27 @@ trait MesosSchedulerUtils extends Logging { } fwInfoBuilder.setHostname(Option(conf.getenv("SPARK_PUBLIC_DNS")).getOrElse( conf.get(DRIVER_HOST_ADDRESS))) + conf.getOption("spark.mesos.principal.file") --- End diff -- I agree. I don't think it's necessary to put the principal in a file, just the secret.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org