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

    https://github.com/apache/spark/pull/19272#discussion_r148013037
  
    --- Diff: 
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
 ---
    @@ -233,10 +249,15 @@ class CoarseGrainedSchedulerBackend(scheduler: 
TaskSchedulerImpl, val rpcEnv: Rp
             context.reply(true)
     
           case RetrieveSparkAppConfig =>
    +        val tokens = if (renewableDelegationTokens.isDefined) {
    +          Some(renewableDelegationTokens.get.credentials)
    +        } else {
    +          None
    +        }
             val reply = SparkAppConfig(
               sparkProperties,
               SparkEnv.get.securityManager.getIOEncryptionKey(),
    -          hadoopDelegationCreds)
    +          tokens)
    --- End diff --
    
    could make `SparkAppConfig` just take a `RenewableDelegationTokens` object? 


---

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

Reply via email to