[
https://issues.apache.org/jira/browse/CALCITE-2736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17613594#comment-17613594
]
Stamatis Zampetakis commented on CALCITE-2736:
----------------------------------------------
As part of this change the following method was modified (a new parameter was
added) and during upgrade to 1.28.0 there were compilation errors cause there
were callers to this method. This is a breaking change and should be part of
the release notes.
{code:java}
protected static boolean reduceExpressions(RelNode rel, List<RexNode> expList,
RelOptPredicateList predicates, boolean unknownAsFalse,
boolean matchNullability)
{code}
Reminder: usually we follow the telescopic pattern; first deprecate the old
method and add new method with new parameter.
> 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
> Fix For: 1.28.0
>
> Time Spent: 2h
> 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.20.10#820010)
