@twalthr You're right, ReduceExpressionRule.CALC should add to logical opt 
phase, but I tried it and it result to java.lang.OutOfMemoryError when I test 
`testReduceDeterministicUDF`. I believe the calc rules (`FilterCalcMergeRule, 
ProjectCalcMergeRule, FilterToCalcRule, ProjectToCalcRule, CalcMergeRule`) 
interact with each other.
If adding a new phase after, there's only calc node, the search space is quite 
small than in logical opt phase.
I think adding phase is necessary as the number of rule increasing.

[ Full content available at: https://github.com/apache/flink/pull/6455 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to