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/06/24 06:43:27 UTC

[GitHub] [flink-table-store] JingsongLi commented on pull request #173: [FLINK-28219] Refactor TableStoreCatalog: Introduce a dedicated Catalog for table store

JingsongLi commented on PR #173:
URL: https://github.com/apache/flink-table-store/pull/173#issuecomment-1165255264

   > Q1: What are the criteria for choosing which interface not to implement. E.g. createDatabase is unsupported, which is confusing to me.
   
   I was previously considering database creation all the way to lazy, but exposing it is more comprehensive.
   So yes, I will change this.
   
   > O2: It seems like FlinkCatalog serves as a wrapper to interact with Flink. Does it only accept the 3-rd party ecosystem's database as the inner impl?
   
   See above comment, we should change back to support the previous functionality


-- 
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