[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9114 --- 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-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-15 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148450238 LGTM, I'll fix the nit while merging. --- 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

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-15 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/9114#discussion_r42148461 --- Diff: launcher/src/test/java/org/apache/spark/launcher/SparkSubmitCommandBuilderSuite.java --- @@ -80,7 +82,7 @@ public void testCliParser() throws Except

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread zjffdu
Github user zjffdu commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148289245 @vanzin Failure unit test is fixed, please help review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148286766 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 project

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148286767 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148286676 [Test build #43764 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43764/console) for PR 9114 at commit [`115a8f1`](https://github.

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148272478 [Test build #43764 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43764/consoleFull) for PR 9114 at commit [`115a8f1`](https://gith

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148271590 Merged build started. --- 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-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148271584 Merged build triggered. --- 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 h

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

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

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148254744 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 project

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148254699 [Test build #43751 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43751/console) for PR 9114 at commit [`ee79b20`](https://github.

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148237499 [Test build #43751 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43751/consoleFull) for PR 9114 at commit [`ee79b20`](https://gith

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148236477 Merged build triggered. --- 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 h

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148236494 Merged build started. --- 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-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/9114#discussion_r42020079 --- Diff: launcher/src/test/java/org/apache/spark/launcher/SparkSubmitCommandBuilderSuite.java --- @@ -164,11 +166,16 @@ private void testCmdBuilder(boolean i

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148007347 CC @vanzin I'm no expert but I do see the point that this seems to be checking "is there a non-default props file set" only. --- If your project is set up for it, you ca

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148007176 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 project

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

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

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-148006994 [Test build #43712 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43712/console) for PR 9114 at commit [`c4277c1`](https://github.

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147977489 [Test build #43712 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43712/consoleFull) for PR 9114 at commit [`c4277c1`](https://gith

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147974797 Merged build triggered. --- 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 h

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147974824 Merged build started. --- 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-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

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

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147971031 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 project

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147971027 [Test build #43711 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43711/console) for PR 9114 at commit [`be67f95`](https://github.

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147970919 [Test build #43711 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43711/consoleFull) for PR 9114 at commit [`be67f95`](https://gith

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147969292 Merged build started. --- 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-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9114#issuecomment-147969265 Merged build triggered. --- 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 h

[GitHub] spark pull request: SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Defau...

2015-10-14 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/spark/pull/9114 SPARK-11099 [SPARK SHELL] [SPARK SUBMIT] Default conf property file i… Please help review it. Thanks You can merge this pull request into a Git repository by running: $ git pull https://github