Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r223870371
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/KubernetesConf.scala
---
@@ -61,7 +71,16 @@ private[spark] case class KubernetesConf[T <:
KubernetesRoleSpecificConf](
roleSecretEnvNamesToKeyRefs: Map[String, String],
roleEnvs: Map[String, String],
roleVolumes: Iterable[KubernetesVolumeSpec[_ <:
KubernetesVolumeSpecificConf]],
- sparkFiles: Seq[String]) {
+ sparkFiles: Seq[String],
+ hadoopConfSpec: Option[HadoopConfSpec]) {
+
+ def hadoopConfigMapName: String = s"$appResourceNamePrefix-hadoop-config"
+
+ def kRBConfigMapName: String = s"$appResourceNamePrefix-krb5-file"
--- End diff --
Weird capitalization. `krbConfigMapName`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
