Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/20032#discussion_r158149295 --- 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 -- `conf.get(KUBERNETES_ALLOCATION_BATCH_DELAY)` returns a `Long` if it's a time conf. That's how time configs are expected to be used. You don't need `podAllocationInterval.toLong` later on like you had before.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org