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 2022/12/19 10:18:09 UTC

[GitHub] [flink] twalthr commented on pull request #15137: [FLINK-21396][table] Resolve tables when calling Catalog.createTable/alterTable

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

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