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

    https://github.com/apache/spark/pull/19631#discussion_r150752055
  
    --- Diff: 
core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala
 ---
    @@ -216,7 +216,9 @@ private[spark] object CoarseGrainedExecutorBackend 
extends Logging {
           if (driverConf.contains("spark.yarn.credentials.file")) {
             logInfo("Will periodically update credentials from: " +
               driverConf.get("spark.yarn.credentials.file"))
    -        SparkHadoopUtil.get.startCredentialUpdater(driverConf)
    +        
Utils.classForName("org.apache.spark.deploy.yarn.YarnSparkHadoopUtil")
    --- End diff --
    
    What's the purpose of changing to this way? 


---

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

Reply via email to