[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173538836 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173538635 **[Test build #49872 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49872/consoleFull)** for PR 10723 at commit [`9922ccc`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173525271 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173525273 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173525124 **[Test build #49868 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49868/consoleFull)** for PR 10723 at commit [`3e5a229`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173507314 **[Test build #49872 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49872/consoleFull)** for PR 10723 at commit [`9922ccc`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173494001 **[Test build #49868 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49868/consoleFull)** for PR 10723 at commit [`3e5a229`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173480075 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173480077 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173479942 **[Test build #49854 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49854/consoleFull)** for PR 10723 at commit [`4fc7a60`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173456838 **[Test build #49854 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49854/consoleFull)** for PR 10723 at commit [`4fc7a60`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50361083 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala --- @@ -156,9 +157,10 @@ private[sql] class CatalystQl(val conf: ParserCo

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50361041 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlParser.g --- @@ -2115,7 +2185,7 @@ structType mapType @init { p

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50361063 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystQl.scala --- @@ -140,6 +140,7 @@ private[sql] class CatalystQl(val conf: ParserCon

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50361028 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlParser.g --- @@ -648,6 +653,12 @@ import java.util.HashMap; }

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50360969 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlLexer.g --- @@ -465,7 +467,7 @@ Identifier fragment QuotedI

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50360926 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/ExpressionParser.g --- @@ -518,6 +527,10 @@ principalIdentifier | Quoted

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50360254 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/ExpressionParser.g --- @@ -518,6 +527,10 @@ principalIdentifier | Quoted

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50344839 --- Diff: project/MimaExcludes.scala --- @@ -153,6 +153,9 @@ object MimaExcludes { ProblemFilters.exclude[MissingMethodProblem]("org.apache.sp

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50286554 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/ExpressionParser.g --- @@ -518,6 +527,10 @@ principalIdentifier | Qu

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173146361 OK. The problem is at `org.apache.spark.sql.sources.TableScanSuite`. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173131648 Weird... I can't reproduce that locally. --- 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-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173130156 No I think it's new and related to this change. --- 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 pr

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173129216 @rxin Have you seen this test failure before? I can pass the tests locally. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173126177 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173126176 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173126029 **[Test build #49764 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49764/consoleFull)** for PR 10723 at commit [`559083e`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173109235 **[Test build #49764 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49764/consoleFull)** for PR 10723 at commit [`559083e`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173108390 Why jenkin continues throw exceptions like: [info] Exception encountered when attempting to run a suite with class name: org.apache.spark.sql.execut

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173108412 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 fe

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173088652 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173088651 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173088539 **[Test build #49751 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49751/consoleFull)** for PR 10723 at commit [`559083e`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173068131 **[Test build #49751 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49751/consoleFull)** for PR 10723 at commit [`559083e`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173064266 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 fe

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-173064249 Weird test failure... --- 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-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172892085 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172892080 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172891814 **[Test build #49686 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49686/consoleFull)** for PR 10723 at commit [`559083e`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172859238 **[Test build #49686 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49686/consoleFull)** for PR 10723 at commit [`559083e`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172857283 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 fe

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172826266 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172826265 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172826100 **[Test build #49682 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49682/consoleFull)** for PR 10723 at commit [`559083e`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172791934 **[Test build #49682 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49682/consoleFull)** for PR 10723 at commit [`559083e`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172782982 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172782979 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172782909 **[Test build #49675 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49675/consoleFull)** for PR 10723 at commit [`7e0f218`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172781266 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172781267 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172780910 **[Test build #49669 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49669/consoleFull)** for PR 10723 at commit [`d800c58`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172779262 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 does

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172779308 **[Test build #49675 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49675/consoleFull)** for PR 10723 at commit [`7e0f218`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172779264 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172779212 **[Test build #49673 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49673/consoleFull)** for PR 10723 at commit [`838f701`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50084780 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/ExpressionParser.g --- @@ -518,6 +527,10 @@ principalIdentifier | Quoted

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172775623 **[Test build #49673 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49673/consoleFull)** for PR 10723 at commit [`838f701`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172762190 **[Test build #49669 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49669/consoleFull)** for PR 10723 at commit [`d800c58`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50078090 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -316,7 +316,7 @@ class HiveContext private[hive]( } p

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r50074715 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,84 @@ private[sql] class SparkQl(conf: ParserConf = SimplePar

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49996310 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,83 @@ private[sql] class SparkQl(conf: ParserConf = SimplePar

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49991694 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,81 @@ private[sql] class SparkQl(conf: ParserConf = Simpl

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172503319 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172503316 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172503157 **[Test build #49591 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49591/consoleFull)** for PR 10723 at commit [`1c145eb`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172484340 **[Test build #49591 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49591/consoleFull)** for PR 10723 at commit [`1c145eb`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172483866 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172483867 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49979049 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,83 @@ private[sql] class SparkQl(conf: ParserConf = Simpl

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49978649 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,84 @@ private[sql] class SparkQl(conf: ParserConf = Simpl

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49978129 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -316,7 +316,7 @@ class HiveContext private[hive]( }

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49977545 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlParser.g --- @@ -1756,6 +1795,28 @@ showStmtIdentifier : identif

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49977431 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,81 @@ private[sql] class SparkQl(conf: ParserConf = SimplePar

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49976797 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,81 @@ private[sql] class SparkQl(conf: ParserConf = SimplePar

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49976006 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,81 @@ private[sql] class SparkQl(conf: ParserConf = Simpl

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49975804 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -42,6 +45,81 @@ private[sql] class SparkQl(conf: ParserConf = Simpl

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49975037 --- Diff: sql/catalyst/src/main/antlr3/org/apache/spark/sql/catalyst/parser/SparkSqlParser.g --- @@ -1756,6 +1795,28 @@ showStmtIdentifier : ide

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49974486 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -52,26 +130,30 @@ private[sql] class SparkQl(conf: ParserConf = Sim

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/10723#discussion_r49974415 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkQl.scala --- @@ -52,26 +57,30 @@ private[sql] class SparkQl(conf: ParserConf = Simp

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172460220 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172460210 **[Test build #49585 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49585/consoleFull)** for PR 10723 at commit [`5a6cc4a`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172460219 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-172459978 **[Test build #49585 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49585/consoleFull)** for PR 10723 at commit [`5a6cc4a`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-171168853 Because SQLContext still uses DDLParser, looks like I can't simply remove `describeTable` command from DDLParser. So I can't gradually migrate these three commands in th

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-171045847 cc @hvanhovell @viirya can you rename the title to "migrate describe table parsing to ..." --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-170870270 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-170870144 **[Test build #49237 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49237/consoleFull)** for PR 10723 at commit [`f886f67`](https://g

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-170870268 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 projec

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10723#issuecomment-170864084 **[Test build #49237 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49237/consoleFull)** for PR 10723 at commit [`f886f67`](https://gi

[GitHub] spark pull request: [SPARK-12689][SQL] Migrate DDL parsing to the ...

2016-01-12 Thread viirya
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/10723 [SPARK-12689][SQL] Migrate DDL parsing to the newly absorbed parser: describe command JIRA: https://issues.apache.org/jira/browse/SPARK-12689 DDLParser processes three commands: createTable

<    1   2