You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2021/10/21 14:07:07 UTC

[GitHub] [flink] twalthr edited a comment on pull request #17522: [FLINK-24462][table] Introduce CastRule interface to reorganize casting code

twalthr edited a comment 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 (at least from the outside).


-- 
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-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org