Github user ArtRand commented on the issue:

    https://github.com/apache/spark/pull/19272
  
    @vanzin Thanks for the continued review. I addressed the comments you left. 
PTAL.
    
    It seems simplest and more robust to keep the serialized `Credentials` 
within `CoarseGrainedSchedulerBackend`. The change is that now the renewal time 
is also kept along side. I tried to move as much duplicate code to 
`HadoopDelegationTokenManager` which remains a "toolbox" as before and doesn't 
retain much state. The abstract class `HadoopCredentialRenewer` is now part of 
the scheduler backend also. I refrained from changing too much of the YARN 
specific code. 


---

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

Reply via email to