[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-26 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-151078262 Merged to master/1.5 (after removing that blank line) --- 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-5966][WIP]

2015-10-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9220 --- 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-5966][WIP]

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

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-23 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150669670 LGTM, will merge once tests pass. --- 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-5966][WIP]

2015-10-23 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150669570 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 this

[GitHub] spark pull request: [SPARK-5966][WIP]

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

[GitHub] spark pull request: [SPARK-5966][WIP]

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

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-23 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/9220#discussion_r42904851 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala --- @@ -266,6 +266,7 @@ object SparkSubmit { } } +

[GitHub] spark pull request: [SPARK-5966][WIP]

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

[GitHub] spark pull request: [SPARK-5966][WIP]

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

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150698354 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-5966][WIP]

2015-10-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150138610 Can one of the admins verify this patch? --- 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-5966]

2015-10-22 Thread kevinyu98
GitHub user kevinyu98 opened a pull request: https://github.com/apache/spark/pull/9220 [SPARK-5966] You can merge this pull request into a Git repository by running: $ git pull https://github.com/kevinyu98/spark working_on_spark-5966 Alternatively you can review and apply these

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150178770 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

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150179707 **[Test build #1938 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1938/consoleFull)** for PR 9220 at commit

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/9220#discussion_r42811834 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/impatient.sc --- @@ -0,0 +1 @@ +1+1; --- End diff -- Hello

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/9220#discussion_r42803833 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala --- @@ -266,6 +266,11 @@ object SparkSubmit { } } +

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/9220#discussion_r42811331 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/impatient.sc --- @@ -0,0 +1 @@ +1+1; --- End diff -- Why

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/9220#discussion_r42813223 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala --- @@ -266,6 +266,11 @@ object SparkSubmit { } } +

[GitHub] spark pull request: [SPARK-5966][WIP]

2015-10-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9220#issuecomment-150209795 **[Test build #1938 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1938/consoleFull)** for PR 9220 at commit