Skip to content

Commit

Permalink
Merge pull request apache#9 from ringtail/bugfix/executor-node-name
Browse files Browse the repository at this point in the history
add default node name
  • Loading branch information
ringtail committed Apr 21, 2020
2 parents e6b81d8 + ae1ca6b commit 2814e60
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ private[spark] class BasicDriverFeatureStep(
private val driverLimitCores = conf.get(KUBERNETES_DRIVER_LIMIT_CORES)

// node name of driver
private val driverNodeName = conf.get(KUBERNETES_DRIVER_NODE_NAME)
private val driverNodeName = conf.get(KUBERNETES_DRIVER_NODE_NAME).getOrElse("")

// Memory settings
private val driverMemoryMiB = conf.get(DRIVER_MEMORY)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ private[spark] class BasicExecutorFeatureStep(
private val executorLimitCores =
kubernetesConf.get(KUBERNETES_EXECUTOR_LIMIT_CORES)

private val executorNodeName = kubernetesConf.get(KUBERNETES_EXECUTOR_NODE_NAME)
private val executorNodeName = kubernetesConf.get(KUBERNETES_EXECUTOR_NODE_NAME).getOrElse("")

override def configurePod(pod: SparkPod): SparkPod = {
val name =
Expand Down

0 comments on commit 2814e60

Please sign in to comment.