[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-14 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/13638 @marmbrus I sent a backport pr #13668. But is `branch-1.6` build broken now? Looks working on #13630. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-14 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/13638 @marmbrus Thank you for merging this. Sure, I'll send a backport pr for `branch-1.6`. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-14 Thread marmbrus
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/13638 Hmmm, does not apply cleanly to 1.6. @ueshin if you have time it might be nice to backport. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-14 Thread marmbrus
Github user marmbrus commented on the issue: https://github.com/apache/spark/pull/13638 Yeah, sounds reasonable. Merging to master, 2.0 and 1.6. --- 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 issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-14 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/13638 LGTM, @marmbrus should be backport it to 1.6? --- 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 issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13638 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60459/ Test PASSed. ---

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13638 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 does not have this feature

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13638 **[Test build #60459 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60459/consoleFull)** for PR 13638 at commit

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13638 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60458/ Test PASSed. ---

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13638 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 does not have this feature

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13638 **[Test build #60458 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60458/consoleFull)** for PR 13638 at commit

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/13638 @marmbrus It seems this is not a regression from `1.6`. The test modified for `1.6` also fails with the same error. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13638 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60451/ Test PASSed. ---

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13638 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 does not have this feature

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13638 **[Test build #60451 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60451/consoleFull)** for PR 13638 at commit

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13638 **[Test build #60459 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60459/consoleFull)** for PR 13638 at commit

[GitHub] spark issue #13638: [SPARK-15915][SQL] Logical plans should use canonicalize...

2016-06-13 Thread ueshin
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/13638 Updated pr title, description and a test name. --- 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