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

Yingyu commented on CALCITE-2736:
---------------------------------

[~vladimirsitnikov]Thanks for your help in review and your comment. I agree 
with you that in general we should avoid negative naming. I've updated the 
naming using `treatDynamicCallsAsConstants` as suggested.
Regarding the use case the direct usage would be to reduce query_user as part 
of planning despite it being a dynamic function.

> ReduceExpressionsRule never reduces dynamic expressions but this should be 
> configurable
> ---------------------------------------------------------------------------------------
>
>                 Key: CALCITE-2736
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2736
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Jacques Nadeau
>            Assignee: Jacques Nadeau
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
>
> There are situations where it is helpful to reduce dynamic SqlCalls. Right 
> now, the ReduceExpressionsRule always avoids doing this. We should enhance 
> the rule so this can be configurable depending on where in planning the rule 
> is used.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to