[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

2014-11-14 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3122#issuecomment-63162815 Ok this makes sense. I'm merging in master branch-1.2. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

2014-11-14 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3122#issuecomment-63162821 Thanks a lot! --- 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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

2014-11-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3122 --- 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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

2014-11-10 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/3122#issuecomment-62432990 Actually I don't hit any issues caused by not setting connection timeout but the host trying to connect to a host wait for establishing forever when server side is hang.

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

2014-11-09 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/3122#issuecomment-62320949 This looks good, but could also explain what necessitates this change? Did you observe some error? If nothing else, just putting the error you observed in the JIRA

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

2014-11-05 Thread sarutak
GitHub user sarutak opened a pull request: https://github.com/apache/spark/pull/3122 [SPARK-4260] Httpbroadcast should set connection timeout. Httpbroadcast sets read timeout but doesn't set connection timeout. You can merge this pull request into a Git repository by running:

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

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

[GitHub] spark pull request: [SPARK-4260] Httpbroadcast should set connecti...

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