Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/20032#discussion_r158563300
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/KubernetesClusterSchedulerBackend.scala
---
@@ -86,7 +86,7 @@ private[spark] class KubernetesClusterSchedulerBackend(
private val initialExecutors =
SchedulerBackendUtils.getInitialTargetExecutorNumber(conf)
- private val podAllocationInterval =
conf.get(KUBERNETES_ALLOCATION_BATCH_DELAY)
+ private val podAllocationInterval =
conf.getTimeAsMs(KUBERNETES_ALLOCATION_BATCH_DELAY.key)
--- End diff --
I (incorrectly) assumed KUBERNETES_ALLOCATION_BATCH_DELAY was a String, and
not ConfigEntry.
@vanzin's suggestion is much more elegant in comparison.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
