[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-05-11 Thread xwu0226
Github user xwu0226 closed the pull request at: https://github.com/apache/spark/pull/12579 --- 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-14346][SQL] Show Create Table (Native)

2016-05-11 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-218549665 @liancheng Thank you for the detail explanation!! Yeah. if the goal is to make sure Spark SQL can handle the generated DDL, then, we need to miss some hive features

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-05-11 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-218379687 Hey @xwu0226, sorry that I didn't explain why I opened another PR for the same issue, was in code rush for 2.0... So one of the considerations for all the

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-05-10 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-218238618 @srowen Yes, for datasource table. This PR also includes the work for hive syntax DDL too. I see #12781 mentions that there will be followup PR taking care of the hive

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-05-10 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-218192270 @xwu0226 I think this is superseded by https://github.com/apache/spark/pull/12781 ? --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-216763938 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-14346][SQL] Show Create Table (Native)

2016-04-27 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-215303758 @yhuai @liancheng , I see PR [#12734](https://github.com/apache/spark/pull/12734) takes care of the PARTITIONED BY and CLUSTERED BY (with SORTED BY) clause for CTAS

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-04-25 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-214553367 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

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-04-25 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-214472079 @liancheng Thanks for triggering the test! I am looking into the test failure. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-214466178 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-14346][SQL] Show Create Table (Native)

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

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

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

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

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

[GitHub] spark pull request: [SPARK-14346][SQL] Show Create Table (Native)

2016-04-25 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-214419087 test 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-14346][SQL] Show Create Table (Native)

2016-04-21 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-213126393 @yhuai @andrewor14 Thanks! --- 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-14346][SQL] Show Create Table (Native)

2016-04-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12579#issuecomment-213032974 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-14346][SQL] Show Create Table (Native)

2016-04-21 Thread xwu0226
GitHub user xwu0226 opened a pull request: https://github.com/apache/spark/pull/12579 [SPARK-14346][SQL] Show Create Table (Native) This is a rebased version of [#12132](https://github.com/apache/spark/pull/12132) and [#12406](https://github.com/apache/spark/pull/12406)