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

    https://github.com/apache/spark/pull/20327#discussion_r171381824
  
    --- Diff: 
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala
 ---
    @@ -123,6 +123,10 @@ class YarnClusterSuite extends BaseYarnClusterSuite {
         checkResult(finalState, result)
       }
     
    +  test("yarn-cluster should use NM's public address instead  of 
SPARK_LOCAL_*") {
    +    testBasicYarnApp(false, conf = 
Map("spark.yarn.appMasterEnv.SPARK_LOCAL_IP" -> "1.1.1.1"))
    --- End diff --
    
    1.1.1.1 is a valid IP address (try it out, ping it!); you should probably 
use something invalid here instead (like "0.1.1.1", which you shouldn't be able 
to use as a target IP address).


---

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

Reply via email to