chenqin commented on PR #21728: URL: https://github.com/apache/flink/pull/21728#issuecomment-1396541490
@luoyuxia please help take a look, here is a bit rationale - parser should be PublicEvolving interface while both Flink and hive has own internal implementation. so hive connector maintainer less worry about Flink planner changes - PlannerQueryOperation should keep internal in both table-planner as well as hive-connector so hive connector can have full control and evolve without worry how Flink planner PlannerQueryOperation evolve - PlannerContext is simple enough util can be PublicEvolving -- 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]
