Aaaaaaron commented on pull request #2253: URL: https://github.com/apache/calcite/pull/2253#issuecomment-724407062
> I have a bad feeling about this change. It adds a lot of new code and does not leverage existing code (e.g. reasoning based on predicates). It is going to be a nightmare to maintain. Hi, @julianhyde, what do you mean "does not leverage existing code(e.g. reasoning based on predicates)"? Current I just add a utility to simplify the and/or condition, it belongs to CALCITE_4375, and later we can add this logic to RexSimplify's main process. In fact, the code is not so much, the real code is no more than 20 lines and there's UT to guarantee this logic. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
