[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-04-24 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-95879508 @chenghao-intel ok. --- 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-5212][SQL] Add support of schema-less, ...

2015-04-24 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-95857854 I just file a jira issue, https://issues.apache.org/jira/browse/SPARK-7119. @viirya can you help on investigate this? --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4014 --- 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-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-72546189 Thanks! Merged to 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 have

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-72465789 @marmbrus I did some refactoring for the comments. It should be better now. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23932224 --- Diff: sql/hive/v0.12.0/src/main/scala/org/apache/spark/sql/hive/Shim12.scala --- @@ -241,8 +241,14 @@ private[hive] object HiveShim {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23932391 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,18 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23932681 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,18 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-02 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23932592 --- Diff: sql/hive/v0.12.0/src/main/scala/org/apache/spark/sql/hive/Shim12.scala --- @@ -241,8 +241,14 @@ private[hive] object HiveShim {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904350 --- Diff: sql/hive/v0.13.1/src/main/scala/org/apache/spark/sql/hive/Shim13.scala --- @@ -395,9 +397,26 @@ private[hive] object HiveShim {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904492 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala --- @@ -167,8 +167,10 @@ private[hive] trait HiveStrategies {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-72402698 Thanks for working on this! It would be great if this could be updated soon so we can include it in 1.3. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904457 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala --- @@ -167,8 +167,10 @@ private[hive] trait HiveStrategies {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904448 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,29 +628,71 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904432 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,29 +628,71 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904329 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -33,7 +33,8 @@ import org.apache.spark.sql.catalyst.plans._ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904318 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +69,205 @@ case class ScriptTransformation(

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904323 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,18 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904386 --- Diff: sql/hive/v0.12.0/src/main/scala/org/apache/spark/sql/hive/Shim12.scala --- @@ -241,8 +241,14 @@ private[hive] object HiveShim {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904416 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,18 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23904403 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,18 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23910289 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,18 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-02-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23910154 --- Diff: sql/hive/v0.12.0/src/main/scala/org/apache/spark/sql/hive/Shim12.scala --- @@ -241,8 +241,14 @@ private[hive] object HiveShim {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-31 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-72315535 @rxin I have added the explanation for this feature. Would you have time to review this pr and see if it is ok to merge? Thanks! --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71835068 @chenghao-intel Thanks for comments. I refactor the codes for these issues. For reusing Hive existing codes, I check the links you mentioned. However, the codes

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71811339 [Test build #26222 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26222/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71819461 [Test build #26222 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26222/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71878029 [Test build #26232 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26232/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71947781 [Test build #26268 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26268/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71943344 [Test build #26268 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26268/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71942858 Upload Hive golden answer files. --- 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-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71949295 [Test build #26273 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26273/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71956530 Can you explain in the PR what is schema-less delimiter? --- 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-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71957621 [Schema-less Map-reduce Scripts](https://cwiki.apache.org/confluence/display/Hive/LanguageManual+Transform#LanguageManualTransform-Schema-lessMap-reduceScripts) is a

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71956176 [Test build #26273 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26273/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71956296 @rxin Would you like to take a look at this too and see if it is ready to merge? Thanks. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71885771 [Test build #26232 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26232/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-26 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23516387 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +78,176 @@ case class ScriptTransformation(

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-26 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23516234 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +78,176 @@ case class ScriptTransformation(

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-25 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23510266 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +78,176 @@ case class

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-25 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23510387 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveQuerySuite.scala --- @@ -327,7 +327,127 @@ class HiveQuerySuite extends

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-25 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23509873 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/ScriptTransformation.scala --- @@ -25,9 +25,24 @@ import

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-25 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23510200 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,29 +627,71 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-25 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23510355 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +78,176 @@ case class

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-25 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71406585 @viirya the `SerDe` and `RowFormat` is quite headache problem, probably we'd better keep reusing the Hive code as much as possible, and also keep it more

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-23 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23441532 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala --- @@ -165,8 +165,20 @@ private[hive] trait HiveStrategies {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-23 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23441619 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveQuerySuite.scala --- @@ -327,7 +327,49 @@ class HiveQuerySuite extends

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-23 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71179511 @chenghao-intel thanks for review. New commits are added for that. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23428628 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,21 +627,56 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23428921 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +82,176 @@ case class ScriptTransformation(

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23429683 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +82,176 @@ case class

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23424723 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,21 +627,56 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23424933 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,21 +627,56 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23425082 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveQuerySuite.scala --- @@ -327,7 +327,49 @@ class HiveQuerySuite extends

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23425650 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,21 +627,56 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-71133818 @viirya it's great to have the feature, I have some general comments on this, let's see how to improve that. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23424312 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala --- @@ -165,8 +165,20 @@ private[hive] trait HiveStrategies {

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23425029 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -627,21 +627,56 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23425907 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -649,7 +684,9 @@

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-22 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4014#discussion_r23426344 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/ScriptTransformation.scala --- @@ -53,28 +82,176 @@ case class

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70470259 [Test build #25756 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25756/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70469836 [Test build #25756 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25756/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70473932 [Test build #25758 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25758/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70481455 [Test build #25758 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25758/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70470261 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70411540 [Test build #25723 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25723/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70411565 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70411564 [Test build #25723 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25723/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70412261 [Test build #25724 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25724/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70421242 [Test build #25729 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25729/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70414868 [Test build #25724 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25724/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70424085 [Test build #25729 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25729/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70370192 [Test build #25703 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25703/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70372569 [Test build #25703 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25703/consoleFull) for PR 4014 at commit

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70292349 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70300945 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

2015-01-16 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70304683 test again. --- 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-5212][SQL] Add support of schema-less, ...

2015-01-16 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70305074 Jenkins, ok to test. --- 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-5212][SQL] Add support of schema-less, ...

2015-01-16 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4014#issuecomment-70305460 Failed by connection error. Please test again. --- 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-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

[GitHub] spark pull request: [SPARK-5212][SQL] Add support of schema-less, ...

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

  1   2   >