You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Vladimir Sitnikov (JIRA)" <ji...@apache.org> on 2015/02/18 23:57:12 UTC

[jira] [Comment Edited] (CALCITE-600) Create UnionFactory and use in rules containing Union operator

    [ https://issues.apache.org/jira/browse/CALCITE-600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14326680#comment-14326680 ] 

Vladimir Sitnikov edited comment on CALCITE-600 at 2/18/15 10:56 PM:
---------------------------------------------------------------------

[~jcamachorodriguez], can you please clarify who creates HiveUnion in the first place?
Can't you perform basic optimizations in {{Logical*}} relations?

{quote}HiveUnion operator that extends Union. I would like to use the UnionMerge rule in Hive. Without this extension, the only solution is to extend the rule and rewrite the match (and possibly transform{quote}
I do not follow you. UnionMergeRule operates on LogicalUnions. Eventually (I guess) those LogicalUnions lower to HiveUnion.
UnionMergeRule will not fire for HiveUnion unless your HiveUnion extends LogicalUnion (however I see little sense in it).


was (Author: vladimirsitnikov):
[~jcamachorodriguez], can you please clarify who creates HiveUnion?
Can't you perform basic optimizations in {{Logical*}} relations?

{quote}HiveUnion operator that extends Union. I would like to use the UnionMerge rule in Hive. Without this extension, the only solution is to extend the rule and rewrite the match (and possibly transform{quote}
I do not follow you. UnionMergeRule operates on LogicalUnions. Eventually those LogicalUnions lower to HiveUnion.
It will not fire for HiveUnion unless your HiveUnion extends LogicalUnion (however I see little sense in it).

> Create UnionFactory and use in rules containing Union operator
> --------------------------------------------------------------
>
>                 Key: CALCITE-600
>                 URL: https://issues.apache.org/jira/browse/CALCITE-600
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Jesus Camacho Rodriguez
>            Assignee: Jesus Camacho Rodriguez
>
> Union related rules match on {{LogicalUnion}} and use this implementation to create the new Union operators. As it is done for other rules, a Union factory can be created and used by them, so subclasses that extend Union can use the rules.



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