[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100127383 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100125476 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-05-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100125531 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

2015-05-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100128735 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-05-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100128741 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100172582 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5744 --- 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-7133][SQL] Implement struct, array, and...

2015-05-08 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100325764 Thanks! Merged to master and 1.4. --- 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-7133][SQL] Implement struct, array, and...

2015-05-07 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-100054122 Other than naming and documentation this LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29898483 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/types/DataType.scala --- @@ -43,7 +43,7 @@ abstract class DataType { /** * Enables

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29897197 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Column.scala --- @@ -68,6 +68,16 @@ class Column(protected[sql] val expr: Expression) extends Logging

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29897799 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,206 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99393385 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99346016 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99352944 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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99353402 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99415256 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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99415506 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99458296 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99415467 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99433997 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99458272 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99519259 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99318206 @cloud-fan mind bringing this up to date? I will ask @marmbrus to review this soon. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99323876 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-05-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-99323881 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97683034 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677458 I will let @marmbrus take a look at this tomorrow. Meantime, can you add the apply method and Python getitem method? Thanks. --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677730 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-04-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677728 @rxin , already done :) --- 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-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97683042 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97677743 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

2015-04-30 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29406389 --- Diff: python/pyspark/sql/dataframe.py --- @@ -1166,7 +1166,7 @@ def __init__(self, jc): # container operators __contains__ =

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97676722 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97707303 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97720821 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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-9774 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97721449 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97721437 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97721148 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97317414 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315450 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315876 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,199 @@ +/* + * Licensed to the

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97355988 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97317347 Jenkins, 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97415055 Jenkins 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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97420520 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97420474 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97356009 Merged build started. --- 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-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315533 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97317379 Jenkins, add to whitelist. --- 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97317409 Merged build triggered. --- 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97316919 The failed test cases looks unrelated, retest? --- 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-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315973 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315738 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97376166 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97338806 The changes looks pretty good. Left comments mostly about documentation, naming, and error messages. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

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

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5744#issuecomment-97314001 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 project

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315596 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala --- @@ -203,5 +210,5 @@ case class UnresolvedGetField(child:

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315432 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala --- @@ -203,5 +210,5 @@ case class UnresolvedGetField(child:

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29315509 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/GetField.scala --- @@ -0,0 +1,199 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-7133][SQL] Implement struct, array, and...

2015-04-29 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5744#discussion_r29314881 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala --- @@ -203,5 +210,5 @@ case class UnresolvedGetField(child:

  1   2   >