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

    https://github.com/apache/spark/pull/19272#discussion_r146946137
  
    --- Diff: 
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
 ---
    @@ -213,6 +216,24 @@ private[spark] class 
MesosCoarseGrainedSchedulerBackend(
           sc.conf.getOption("spark.mesos.driver.frameworkId").map(_ + suffix)
         )
     
    +    // check that the credentials are defined, even though it's likely 
that auth would have failed
    +    // already if you've made it this far
    +    if (principal != null && currentHadoopDelegationTokens.isDefined) {
    +      logDebug(s"Principal found ($principal) starting token renewer")
    +      // The renewal time is ignored when getting the initial delegation 
tokens
    +      // (CoarseGrainedSchedulerBackend.scala:getHadoopDelegationCreds), 
so we get the renewal
    +      // time here and schedule a thread to renew them.
    +      val renewalTime =
    --- End diff --
    
    But do note that you still can't start the token renewer automatically, 
since YARN still has its own renewer and things would get mixed up.


---

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

Reply via email to