You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Jesus Camacho Rodriguez (JIRA)" <ji...@apache.org> on 2016/03/08 11:09:40 UTC

[jira] [Resolved] (CALCITE-1129) Extend JoinUnionTransposeRule to match Union instead of LogicalUnion

     [ https://issues.apache.org/jira/browse/CALCITE-1129?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jesus Camacho Rodriguez resolved CALCITE-1129.
----------------------------------------------
       Resolution: Fixed
    Fix Version/s: 1.7.0

Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/50c4b06. Thanks [~vkalavri]!

> Extend JoinUnionTransposeRule to match Union instead of LogicalUnion
> --------------------------------------------------------------------
>
>                 Key: CALCITE-1129
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1129
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Vasia Kalavri
>            Assignee: Julian Hyde
>            Priority: Minor
>             Fix For: 1.7.0
>
>
> The {{JoinUnionTransposeRule}} is fired if one of the operands is a {{Union}}, but inside the {{onMatch()}} method, the code checks if one of the operands is an instance of {{LogicalUnion}}. External implementations would benefit from changing the rule to match {{Union}} instead.
> The dev mailing list discussion can be found [here|https://mail-archives.apache.org/mod_mbox/calcite-dev/201603.mbox/%3CCAJZ2dcWt3zp5KMCRWPjmDoeWyFKi3AM_NEoQb-9ZD5sRFNSz7Q%40mail.gmail.com%3E].



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)