Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19272#discussion_r146435148
  
    --- Diff: 
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
 ---
    @@ -159,6 +159,12 @@ class CoarseGrainedSchedulerBackend(scheduler: 
TaskSchedulerImpl, val rpcEnv: Rp
             scheduler.getExecutorsAliveOnHost(host).foreach { exec =>
               killExecutors(exec.toSeq, replace = true, force = true)
             }
    +
    +      case UpdateDelegationTokens(tokens) =>
    +        // Update the driver's delegation tokens in case new executors are 
added later.
    +        currentHadoopDelegationTokens = Some(tokens)
    +        executorDataMap.values.foreach { ed =>
    +          ed.executorEndpoint.send(UpdateDelegationTokens(tokens)) }
    --- End diff --
    
    `}` goes in next line.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to