[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2016-01-23 Thread blbradley
Github user blbradley commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-174248297 @srowen httpclient 4.5.* depends on versions of httpcore 4.4.* --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163433547 The failure seems unrelated to my change, so I will ask Jenkins to retest. Also, I am not sure if I have the right permissions to trigger a new build and I'd

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163434873 Jenkins add to whitelist --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163437588 **[Test build #47456 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47456/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163462155 LGTM. Thanks @markgrover! --- 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-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163461265 **[Test build #47456 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47456/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163445940 Thanks, @srowen! --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163433567 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163461936 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163461933 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163467443 I'm going to update the PR title and description in the final commit so that they're no longer WIP / so verbose. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163467327 Tested locally and confirmed that these tests still run fine in SBT, so I'm going to merge this to master and branch-1.6. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163468789 Thanks so much, Josh! --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9876 --- 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-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163091265 **[Test build #47399 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47399/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163095557 **[Test build #47399 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47399/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163095565 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-08 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163091048 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-12-08 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-163091025 Thanks @JoshRosen For now, I have added the mysql and postgresql drivers as dependencies and the tests pass now. We just have to make sure that the CI environment

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-23 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-159065048 FWIW it would probably be easier to produce a new release of the Docker client which properly relocates and shades dependencies; see my comments + discussion at

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-22 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158758482 Should the client/core versions match up? and be declared as one global version? I think using the latest of these is generally a Good Thing, but I know so many

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-21 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158651416 OK, I will do that. I am afk for some time but will take care of this when I am back. Thanks. On Nov 20, 2015 10:12 PM, "Josh Rosen"

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158578006 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158577984 **[Test build #46462 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46462/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158578537 **[Test build #46467 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46467/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158593498 My hunch is that you'll have to declare an explicit dependency on those drivers, since I think that there's a difference between Maven and SBT handling of test-jars'

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158588669 Thanks Josh. So, I have a good news and bad news. The good news is that the maven/dependency related issues have been resolved. The bad news is that the

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158586420 **[Test build #46467 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46467/consoleFull)** for PR 9876 at commit

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158586429 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

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

[GitHub] spark pull request: [SPARK-11796] [test-maven] [WIP] Fixing httpcl...

2015-11-20 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9876#issuecomment-158578048 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