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

Julian Hyde commented on CALCITE-1653:
--------------------------------------

[~rusanu], One more thing. The rex package should not depend on the plan 
package (except for RelOptUtils). So, can you rename {{interface 
RelOptPlanner.Executor}} to {{org.apache.calcite.rex.RexExecutor}}. If you 
think it will help compatibility, deprecate {{RelOptPlanner.Executor}} and have 
it as a base class for the new {{RexExecutor}}.

> Allow for custom RelOptPlanner.Executor in RexUtil.simplify
> -----------------------------------------------------------
>
>                 Key: CALCITE-1653
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1653
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Remus Rusanu
>            Assignee: Remus Rusanu
>         Attachments: CALCITE-1653.00.patch
>
>
> {{RexUtil.simplify}} (and derivatives) use a default {{EXECUTOR}} for 
> evaluating literals and expression in {{simplifyCast}}. This causes value 
> differences from Hive, triggering issues like CALCITE-1650, CALCITE-1651. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to