spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vanzin <...@git.apache.org>
Subject [GitHub] spark pull request #20657: [SPARK-23361][yarn] Allow AM to restart after ini...
Date Mon, 05 Mar 2018 23:17:30 GMT
Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20657#discussion_r172362293
  
    --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala ---
    @@ -144,7 +145,8 @@ class SparkHadoopUtil extends Logging {
       private[spark] def addDelegationTokens(tokens: Array[Byte], sparkConf: SparkConf) {
         UserGroupInformation.setConfiguration(newConfiguration(sparkConf))
         val creds = deserialize(tokens)
    -    logInfo(s"Adding/updating delegation tokens ${dumpTokens(creds)}")
    +    logInfo("Updating delegation tokens for current user.")
    --- End diff --
    
    That information is already logged by the AM, that's enough for debugging.


---

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


Mime
View raw message