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

    https://github.com/apache/spark/pull/20327#discussion_r171379388
  
    --- Diff: 
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala
 ---
    @@ -79,6 +80,19 @@ private[spark] class ApplicationMaster(args: 
ApplicationMasterArguments) extends
     
       private val yarnConf = new 
YarnConfiguration(SparkHadoopUtil.newConfiguration(sparkConf))
     
    +  if (isClusterMode) {
    +    // this logic replicates the way YARN NM determines the address to 
bind listeners to
    --- End diff --
    
    This explain what it does, but it should be explaining why it does it.


---

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

Reply via email to