Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22362#discussion_r220360201
--- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala ---
@@ -726,7 +726,11 @@ private[spark] object SparkConf extends Logging {
DRIVER_MEMORY_OVERHEAD.key -> Seq(
AlternateConfig("spark.yarn.driver.memoryOverhead", "2.3")),
EXECUTOR_MEMORY_OVERHEAD.key -> Seq(
- AlternateConfig("spark.yarn.executor.memoryOverhead", "2.3"))
+ AlternateConfig("spark.yarn.executor.memoryOverhead", "2.3")),
+ KEYTAB.key -> Seq(
+ AlternateConfig("spark.yarn.keytab", "2.4")),
--- End diff --
This change doesn't make a whole lot of sense without the rest of the k8s
kerberos work, and that change is a lot more controversial to pull into 2.4 at
this point. So I'm leaning towards no.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
