twalthr commented on PR #15137: URL: https://github.com/apache/flink/pull/15137#issuecomment-1357415378
@Aitozi this is the result of a longer internal discussion. The biggest reason is backwards compatibility, the current design allowed to not break catalog implementations. The `Catalog` interface is actually 2-level API (similar to DynamicTableFactory) that should not be exposed to Table API main users. However, `TableEnvironment` exposes a `getCatalog` and when using this API unresolved instances might enter the interface implementation. We should actually deprecate `TableEnvironment.getCatalog` because only the framework should call Catalog implementations, not users. -- 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]
