Github user fhueske commented on the issue:

    https://github.com/apache/flink/pull/2102
  
    This @wuchong, your approach looks good. I also found that the 
`ReduceExpressionRules` had no effect due to the missing `RexExecutor`. 
    
    However, it seems that several tests of `ExpressionITCase` are failing with 
this change. You can verify that the PR does not break the build by locally 
running `mvn clean install`.
    
    In addition, the added test should be changed as sketched in the comment. 
Please let me know if you have questions.
    
    Thanks, Fabian


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to