xiedeyantu commented on PR #4577: URL: https://github.com/apache/calcite/pull/4577#issuecomment-3395445368
> Could be a valid approach. It's aligned with already existing aspects like "deterministic" and "dynamic". Maybe in the long run we could even consider combining these fields (`deterministic`, `dynamic`, `idempotent`, and any other that may come) into a single flag field instead of N booleans. Yes, I think we can continue to integrate or refactor this later. The current PR is mainly intended to share an initial idea. If you'd like me to further improve it, please let me know — I'm not sure if the original author has time to continue working on their PR. -- 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]
