[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153257410 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-02 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153278971 LGTM cc @yhuai --- 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-9034] [SQL] Reflect field names defined...

2015-11-02 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153281867 Thanks! Merging 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-9034] [SQL] Reflect field names defined...

2015-11-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8456 --- 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-9034] [SQL] Reflect field names defined...

2015-11-02 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r43645500 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala --- @@ -53,7 +53,7 @@ trait Generator extends Expression

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153234647 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-9034] [SQL] Reflect field names defined...

2015-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153234640 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153223809 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-9034] [SQL] Reflect field names defined...

2015-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153223667 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-153228819 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-9034] [SQL] Reflect field names defined...

2015-11-02 Thread navis
Github user navis commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r43711653 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala --- @@ -53,7 +53,7 @@ trait Generator extends Expression {

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-152894012 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-152880044 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-9034] [SQL] Reflect field names defined...

2015-11-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-152880038 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-9034] [SQL] Reflect field names defined...

2015-11-01 Thread navis
Github user navis commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r43590635 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1119,7 +1119,8 @@ class DataFrame private[sql]( def explode[A <: Product :

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-152468131 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-9034] [SQL] Reflect field names defined...

2015-10-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-152468109 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-28 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-151741657 LGTM, /cc @liancheng @cloud-fan for taking another look. --- 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-9034] [SQL] Reflect field names defined...

2015-10-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r43223232 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1119,7 +1119,8 @@ class DataFrame private[sql]( def explode[A <:

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-28 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r43237332 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala --- @@ -1119,7 +1119,8 @@ class DataFrame private[sql]( def explode[A <:

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-150997969 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-9034] [SQL] Reflect field names defined...

2015-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-150997975 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-9034] [SQL] Reflect field names defined...

2015-10-25 Thread navis
Github user navis commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r42953994 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -148,7 +148,7 @@ class Analyzer( child match

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-25 Thread navis
Github user navis commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-150997853 Rebased to trunk --- 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-151014681 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-21 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8456#discussion_r42706827 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -148,7 +148,7 @@ class Analyzer(

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-21 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-150091426 I'd like to see this will be fixed as it's a long time bug I know. Can you bring it back? It general looks good to me, but I leave a question also in the code.

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-145419797 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-10-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-145407559 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-9034] [SQL] Reflect field names defined...

2015-10-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-145407569 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-9034] [SQL] Reflect field names defined...

2015-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-134899708 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-9034] [SQL] Reflect field names defined...

2015-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-134899685 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-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-08-26 Thread navis
GitHub user navis opened a pull request: https://github.com/apache/spark/pull/8456 [SPARK-9034] [SQL] Reflect field names defined in GenericUDTF Hive GenericUDTF#initialize() defines field names in a returned schema though, the current HiveGenericUDTF drops these names. We

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

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

[GitHub] spark pull request: [SPARK-9034] [SQL] Reflect field names defined...

2015-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8456#issuecomment-134947919 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