[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202651699 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202651700 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202651485 **[Test build #54384 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54384/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11948 --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202629690 Thanks merging into master. --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202628654 **[Test build #54384 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54384/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/11948#discussion_r57651545 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -209,6 +208,9 @@ private[hive] class HiveQl(conf: ParserConf) extends

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202624752 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202605688 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202605684 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202605002 **[Test build #54350 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54350/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202561979 **[Test build #54350 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54350/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202559116 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202559122 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202558782 **[Test build #54338 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54338/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202527512 **[Test build #54338 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54338/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-28 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-202526659 OK, @yhuai PTAL --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-26 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201934254 @hvanhovell The thought is that it is better to all commands by Spark because asking hive to execute some commands may introduce inconsistent behaviors or failures with

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-26 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201755230 @andrewor14 I thought we passed these on to Hive and would execute these for us. Is this not working at? As for better messaging: You could do some

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201518566 @yhuai if we want any special message we'll need to parse the tokens ourselves. We could do that but it requires more code. Any concrete suggestions in mind for the

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201480224 Thanks for the PR! It will be great if we can make it clear what command is not supported. From the current error message, we only know there are features that we do not

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201468856 updated --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201196520 Sorry for not being clear. I meant the entire exception. I'm wondering if there are any funny error showing up in the that exception. --- If your project is set up for

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201195751 updated --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201181891 never mind it is catalystql. can you put the actual error message thrown into the pr description? --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201181195 who's actually throwing the unsupported exception? --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-25 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201155647 @yhuai this should be straightforward --- 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201120631 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201120629 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201120486 **[Test build #54115 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54115/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201107932 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201107934 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201107743 **[Test build #54119 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54119/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201103741 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-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201103742 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201103651 **[Test build #54122 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54122/consoleFull)** for PR 11948 at commit

[GitHub] spark pull request: [SPARK-14119][SPARK-14120][SPARK-14122][SQL] T...

2016-03-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11948#issuecomment-201086374 **[Test build #54122 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54122/consoleFull)** for PR 11948 at commit