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

    https://github.com/apache/spark/pull/19272#discussion_r139636800
  
    --- Diff: 
core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala
 ---
    @@ -174,6 +178,16 @@ private[spark] class CoarseGrainedExecutorBackend(
     
     private[spark] object CoarseGrainedExecutorBackend extends Logging {
     
    +  private def addDelegationTokens(tokens: Array[Byte], sparkConf: 
SparkConf) {
    --- End diff --
    
    Should we move this to SparkHadoopUtil and re-use 
    methods such as: 
https://github.com/apache/spark/blob/d7b1fcf8f0a267322af0592b2cb31f1c8970fb16/core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala#L131
 ?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to