dawidwys commented on issue #8585: [FLINK-12690][table-api] Introduce a Planner interface URL: https://github.com/apache/flink/pull/8585#issuecomment-498543605 You are right, the `Planner` will need the `CatalogManager` (not only for parse, but also for Table scans). I don't want to add an `init` method though as this is rather bad design to have objects that you can construct but they are unusable unitl some method call. Therefore we should just add the `CatalogManager` to the ctor of the `Planner`.
---------------------------------------------------------------- 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] With regards, Apache Git Services
