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

    https://github.com/apache/spark/pull/22624#discussion_r229059902
  
    --- Diff: 
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
 ---
    @@ -230,7 +242,7 @@ class CoarseGrainedSchedulerBackend(scheduler: 
TaskSchedulerImpl, val rpcEnv: Rp
             val reply = SparkAppConfig(
               sparkProperties,
               SparkEnv.get.securityManager.getIOEncryptionKey(),
    -          fetchHadoopDelegationTokens())
    +          Option(delegationTokens.get()))
    --- End diff --
    
    Actually since I moved the token manager to `CoarseGrainedSchedulerBackend` 
I think I can simplify this and avoid having to trace how the RPC code works to 
prove this is fine. I'll try to update things this afternoon after I finish 
some other stuff.


---

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

Reply via email to