[ 
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)

Reply via email to