[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2015-01-25 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-71416586 @dizzy2 Sorry, thought I responded earlier -- you are quite right. It should be a straightforward fix of adding a `Utils.startServiceOnPort` call to do the same incremen

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2015-01-12 Thread dizzy2
Github user dizzy2 commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-69588907 this fix doesn't work when both executor and driver are running on the same machine (as they are trying to acquire the same port) --- If your project is set up for it, yo

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3688 --- 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 enab

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-18 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-67582457 LGTM, too, so I'm going to merge this into `master` (1.3.0) and `branch-1.2` (1.2.1). Thanks! --- If your project is set up for it, you can reply to this email and ha

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-67246124 LGTM. --- 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

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-67230692 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-67230685 [Test build #24503 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24503/consoleFull) for PR 3688 at commit [`ebd2007`](https://gith

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-67218182 [Test build #24503 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24503/consoleFull) for PR 3688 at commit [`ebd2007`](https://githu

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-67217419 Jenkins, retest this please. --- 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 th

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66928133 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66928132 [Test build #24441 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24441/consoleFull) for PR 3688 at commit [`ebd2007`](https://gith

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66925608 [Test build #24441 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24441/consoleFull) for PR 3688 at commit [`ebd2007`](https://githu

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-14 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66925510 Jenkins, retest this please. --- 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 hav

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66865449 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66865446 [Test build #24430 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24430/consoleFull) for PR 3688 at commit [`ebd2007`](https://gith

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3688#issuecomment-66864203 [Test build #24430 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24430/consoleFull) for PR 3688 at commit [`ebd2007`](https://githu

[GitHub] spark pull request: [SPARK-4837] NettyBlockTransferService should ...

2014-12-12 Thread aarondav
GitHub user aarondav opened a pull request: https://github.com/apache/spark/pull/3688 [SPARK-4837] NettyBlockTransferService should use spark.blockManager.port config This is used in NioBlockTransferService here: https://github.com/apache/spark/blob/master/core/src/main/scala/o