[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-71734642 Ah never mind this is my own pull request. Closing it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-27 Thread rxin
Github user rxin closed the pull request at: https://github.com/apache/spark/pull/4127 --- 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 enable

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-27 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-71711069 I don't have permission, too. I only have `comment` permission here. what's going wrong? --- If your project is set up for it, you can reply to this email an

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-71710163 I don't have permission to do it. Can you click the close button? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-27 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-71709975 ok, please close this one : ) --- 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-5135][SQL] Add support for describe [ex...

2015-01-27 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-71708856 Thanks. Do you mind closing this one? --- 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 no

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-27 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-71707934 Hi, @rxin I created a new PR #4227 to rewrite `this part` and bring everything up-to-date, would you please review it? --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-22 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23359955 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala --- @@ -178,3 +180,34 @@ case class DescribeCommand( child.ou

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-21 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23359441 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala --- @@ -178,3 +180,34 @@ case class DescribeCommand( child.ou

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23276780 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -70,9 +71,16 @@ class HiveContext(sc: SparkContext) extends SQLC

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23276701 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala --- @@ -178,3 +180,34 @@ case class DescribeCommand( child.output.ma

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23276671 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/types/dataTypes.scala --- @@ -300,6 +301,8 @@ case object StringType extends NativeType wit

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23276556 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala --- @@ -178,3 +180,34 @@ case class DescribeCommand( child.ou

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread OopsOutOfMemory
Github user OopsOutOfMemory closed the pull request at: https://github.com/apache/spark/pull/3935 --- 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 featur

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23275712 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/commands.scala --- @@ -178,3 +180,34 @@ case class DescribeCommand( child.output.ma

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23275690 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -70,9 +71,16 @@ class HiveContext(sc: SparkContext) extends SQLContext(

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4127#discussion_r23275660 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/types/dataTypes.scala --- @@ -300,6 +301,8 @@ case object StringType extends NativeType with Prim

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70776973 Yea we can close this one. Will merge that one when tests pass. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70772639 Thanks @rxin Should I close this PR? --- 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-5135][SQL] Add support for describe [ex...

2015-01-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-70771239 [Test build #25859 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25859/consoleFull) for PR 4127 at commit [`b1a0f6a`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4127#issuecomment-70764859 [Test build #25859 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25859/consoleFull) for PR 4127 at commit [`b1a0f6a`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70764410 Thanks. I resolved the conflict and pushed a PR https://github.com/apache/spark/pull/4127 --- If your project is set up for it, you can reply to this email and have your re

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/4127 [SPARK-5135][SQL] Add support for describe [extended] table to DDL in SQLContext This brings https://github.com/apache/spark/pull/3935 up to date and fixes style violations. You can merge this pull

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r23269744 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -70,8 +71,15 @@ class HiveContext(sc: SparkContext) extends SQLContext(sc)

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-20 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r23269711 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -70,8 +71,15 @@ class HiveContext(sc: SparkContext) extends SQLContext(sc)

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70285046 [Test build #25665 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25665/consoleFull) for PR 3935 at commit [`d1689e2`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70273405 [Test build #25665 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25665/consoleFull) for PR 3935 at commit [`d1689e2`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70244538 [Test build #25655 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25655/consoleFull) for PR 3935 at commit [`5b7ae19`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70243136 [Test build #25654 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25654/consoleFull) for PR 3935 at commit [`9d22708`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70237945 Thanks @scwf @rxin, conflicts resolved cleanly, this now up-to-date. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70237708 [Test build #25655 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25655/consoleFull) for PR 3935 at commit [`5b7ae19`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70233294 @OopsOutOfMemory, you need revert unnecessary changes --- 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-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70232782 [Test build #25654 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25654/consoleFull) for PR 3935 at commit [`9d22708`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70230293 @OopsOutOfMemory do like this: git checkout describe git pull apachesparkrepo git fetch apachesparkrepo master:newdescribe git diff newdescribe > describe.pat

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70229207 [Test build #25651 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25651/consoleFull) for PR 3935 at commit [`9efdf35`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70228217 [Test build #25651 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25651/consoleFull) for PR 3935 at commit [`9efdf35`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70220923 Not sure - do you have a backup? Maybe just take a diff and apply the diff on master. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-16 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70220078 @rxin The rebase may have some problems, cause changed 278 files ? how do I revert it ? --- If your project is set up for it, you can reply to this email and hav

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70217508 [Test build #25639 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25639/consoleFull) for PR 3935 at commit [`521bbd7`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70217298 [Test build #25639 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25639/consoleFull) for PR 3935 at commit [`521bbd7`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70199626 Thanks. @rxin I'wll make this up-to-date. --- 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-5135][SQL] Add support for describe [ex...

2015-01-15 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r23059970 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -328,18 +328,19 @@ class SQLContext(@transient val sparkContext: SparkC

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70198837 BTW the data type refactoring PR I was referring to was https://github.com/apache/spark/pull/3958 --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-70198790 Hi @OopsOutOfMemory, I was holding off this one due to the data type refactoring. Now the data type refactoring is in, do you mind bringing it up to date? This is de

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r23059763 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/sources/ddl.scala --- @@ -17,6 +17,11 @@ package org.apache.spark.sql.sources +import

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-15 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r23059758 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -328,18 +328,19 @@ class SQLContext(@transient val sparkContext: SparkContext)

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-13 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69859023 ping /cc @rxin --- 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-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69694874 [Test build #25455 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25455/consoleFull) for PR 3935 at commit [`03a1b6c`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69690973 [Test build #25455 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25455/consoleFull) for PR 3935 at commit [`03a1b6c`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69690119 [Test build #25452 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25452/consoleFull) for PR 3935 at commit [`a2cbf6b`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69689308 [Test build #25452 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25452/consoleFull) for PR 3935 at commit [`a2cbf6b`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69687277 [Test build #25447 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25447/consoleFull) for PR 3935 at commit [`2d159e5`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69686968 [Test build #25447 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25447/consoleFull) for PR 3935 at commit [`2d159e5`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69485874 Updated. Could u review it ? --- 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-5135][SQL] Add support for describe [ex...

2015-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69485494 [Test build #25374 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25374/consoleFull) for PR 3935 at commit [`7d0d1c9`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69485332 [Test build #25373 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25373/consoleFull) for PR 3935 at commit [`b25d32a`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69484328 [Test build #25374 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25374/consoleFull) for PR 3935 at commit [`7d0d1c9`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69484168 [Test build #25373 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25373/consoleFull) for PR 3935 at commit [`b25d32a`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r22764434 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/sources/DDLTestSuit.scala --- @@ -0,0 +1,74 @@ +/* +* Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r22763208 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/sources/DDLTestSuit.scala --- @@ -0,0 +1,74 @@ +/* +* Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69474872 /cc @rxin Here is another public API we should consider standardizing for 1.3. Do we want to have a unified `DESCRIBE` that works for both SQL and Hive tables?

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r22762496 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/sources/DDLTestSuit.scala --- @@ -0,0 +1,74 @@ +/* +* Licensed to the Apache Software Foundati

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r22762492 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/sources/DDLTestSuit.scala --- @@ -0,0 +1,74 @@ +/* +* Licensed to the Apache Software Foundati

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3935#discussion_r22762487 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/sources/ddl.scala --- @@ -77,6 +78,16 @@ private[sql] class DDLParser extends StandardTokenParsers wit

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69455901 [Test build #25359 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25359/consoleFull) for PR 3935 at commit [`e195667`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69454126 [Test build #25359 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25359/consoleFull) for PR 3935 at commit [`e195667`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-09 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69360125 @marmbrus I think it's ok to merge, could u review this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69308515 [Test build #25312 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25312/consoleFull) for PR 3935 at commit [`8f25bd7`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69303311 retest it 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 thi

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69303281 [Test build #25312 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25312/consoleFull) for PR 3935 at commit [`8f25bd7`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69299939 [Test build #25301 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25301/consoleFull) for PR 3935 at commit [`4a24144`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69297563 [Test build #25301 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25301/consoleFull) for PR 3935 at commit [`4a24144`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69297004 [Test build #25297 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25297/consoleFull) for PR 3935 at commit [`80becf3`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69294674 [Test build #25297 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25297/consoleFull) for PR 3935 at commit [`80becf3`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69294135 [Test build #25296 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25296/consoleFull) for PR 3935 at commit [`b5caa6e`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69294092 [Test build #25296 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25296/consoleFull) for PR 3935 at commit [`b5caa6e`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69291456 @scwf Thanks for point it out. 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 pr

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69220158 @OopsOutOfMemory, test failed for hive describe cmd, this is because now hive describe will go into ddlParser not hiveQL ``` new SchemaRDD(this, ddlParser

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69215154 [Test build #25232 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25232/consoleFull) for PR 3935 at commit [`a4489c1`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69208329 [Test build #25232 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25232/consoleFull) for PR 3935 at commit [`a4489c1`](https://githu

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

2015-01-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3935#issuecomment-69204168 [Test build #25231 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25231/consoleFull) for PR 3935 at commit [`9d6861a`](https://gith

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

[GitHub] spark pull request: [SPARK-5135][SQL] Add support for describe [ex...

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

  1   2   >