mihaibudiu commented on PR #4577: URL: https://github.com/apache/calcite/pull/4577#issuecomment-3402386803
I have changed my mind, I don't think this is an ideal design for two reasons: - it is not sustainable to add flags to all functions for every property that may be interesting - simplify is not the right place for all optimizations; this optimization in particular should be done only once, while simplify runs frequently. I think the right way to solve this is through a visitor pattern; the visitor will know which functions are idempotent. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
