[
https://issues.apache.org/jira/browse/FLINK-5570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15939344#comment-15939344
]
Haohui Mai commented on FLINK-5570:
-----------------------------------
I think the APIs of the PR looks good to me overall.
One question -- Does the PR need additional fixes on to make functions like
{{isRegistered}} and {{getRowType}} be aware of databases?
> Support register external catalog to table environment
> ------------------------------------------------------
>
> Key: FLINK-5570
> URL: https://issues.apache.org/jira/browse/FLINK-5570
> Project: Flink
> Issue Type: Sub-task
> Components: Table API & SQL
> Reporter: Kurt Young
> Assignee: jingzhang
>
> This issue aims to support register one or more {{ExternalCatalog}} (which is
> referred in https://issues.apache.org/jira/browse/FLINK-5568) to
> {{TableEnvironment}}. After registration, SQL and TableAPI queries could
> access to tables in the external catalogs without register those tables one
> by one to {{TableEnvironment}} beforehand.
> We plan to add two APIs in {{TableEnvironment}}:
> 1. register externalCatalog
> {code}
> def registerExternalCatalog(name: String, externalCatalog: ExternalCatalog):
> Unit
> {code}
> 2. scan a table from registered catalog and returns the resulting {{Table}},
> the API is very useful in TableAPI queries.
> {code}
> def scan(catalogName: String, tableIdentifier: TableIdentifier): Table
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
