[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-24 Thread lianhuiwang
Github user lianhuiwang commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-68059363 OK. I will close this PR. --- 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

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-24 Thread lianhuiwang
Github user lianhuiwang closed the pull request at: https://github.com/apache/spark/pull/3061 --- 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

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244632 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -39,6 +41,10 @@ final class NioBlockTransferService(conf:

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244663 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -121,8 +132,34 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244677 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -121,8 +132,34 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244702 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -121,8 +132,34 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244710 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -96,6 +102,11 @@ final class NioBlockTransferService(conf:

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244771 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -121,8 +132,34 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22244783 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -121,8 +132,34 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread lianhuiwang
Github user lianhuiwang commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22245434 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -39,6 +41,10 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-68024341 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread lianhuiwang
Github user lianhuiwang commented on a diff in the pull request: https://github.com/apache/spark/pull/3061#discussion_r22245488 --- Diff: core/src/main/scala/org/apache/spark/network/nio/NioBlockTransferService.scala --- @@ -121,8 +132,34 @@ final class

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

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

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

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

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-12-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-68026214 Actually let's close this one. We have already implemented the same functionality in Netty. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-11-02 Thread lianhuiwang
GitHub user lianhuiwang opened a pull request: https://github.com/apache/spark/pull/3061 [SPARK-4195][Core]retry to fetch blocks's result when fetchfailed's reason is connection timeout when there are many executors in a application(example:1000),Connection timeout often occure.

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-11-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-61406820 [Test build #22761 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22761/consoleFull) for PR 3061 at commit

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-11-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-61406896 [Test build #22761 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22761/consoleFull) for PR 3061 at commit

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-61406898 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4195][Core]retry to fetch blocks's resu...

2014-11-02 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3061#issuecomment-61421370 @aarondav we should incorporate this into the new transport thing --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as