twalthr commented on pull request #17522: URL: https://github.com/apache/flink/pull/17522#issuecomment-948652702
Logically they might belong together but only `CastExecutor` is really necessary in common. We should try to keep this module slim. And the concept of `CastRule` and `CastRulePredicate` is a planner concept. In theory, one can fuse `CastRule` and `CastExecutor` into one concept. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
