GitHub user jerryshao opened a pull request:

    https://github.com/apache/spark/pull/8502

    [YARN][Minor] Avoid hard code port number in YarnShuffleService test

    Current port number is fixed as default (7337) in test, this will introduce 
port contention exception, better to change to a random number in unit test.
    
    @squito , seems you're author of this unit test, mind taking a look at this 
fix? Thanks a lot.
    
    ```
    [info] - executor state kept across NM restart *** FAILED *** (597 
milliseconds)
    [info]   org.apache.hadoop.service.ServiceStateException: 
java.net.BindException: Address already in use
    [info]   at 
org.apache.hadoop.service.ServiceStateException.convert(ServiceStateException.java:59)
    [info]   at 
org.apache.hadoop.service.AbstractService.init(AbstractService.java:172)
    [info]   at 
org.apache.spark.network.yarn.YarnShuffleServiceSuite$$anonfun$1.apply$mcV$sp(YarnShuffleServiceSuite.scala:72)
    [info]   at 
org.apache.spark.network.yarn.YarnShuffleServiceSuite$$anonfun$1.apply(YarnShuffleServiceSuite.scala:70)
    [info]   at 
org.apache.spark.network.yarn.YarnShuffleServiceSuite$$anonfun$1.apply(YarnShuffleServiceSuite.scala:70)
    [info]   at 
org.scalatest.Transformer$$anonfun$apply$1.apply$mcV$sp(Transformer.scala:22)
    [info]   at org.scalatest.OutcomeOf$class.outcomeOf(OutcomeOf.scala:85)
    [info]   at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
    [info]   at org.scalatest.Transformer.apply(Transformer.scala:22)
    [info]   at org.scalatest.Transformer.apply(Transformer.scala:20)
    [info]   at org.scalatest.FunSuiteLike$$anon$1.apply(FunSuiteLike.scala:166)
    [info]   at 
org.apache.spark.SparkFunSuite.withFixture(SparkFunSuite.scala:42)
    ...
    ```

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jerryshao/apache-spark avoid-hardcode-port

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/8502.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #8502
    
----
commit bcb93a8fd991f8b242cc837e444e1f84f5e64b62
Author: jerryshao <[email protected]>
Date:   2015-08-28T09:02:30Z

    Avoid hardcode port number in YarnShuffleService test

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to