JingsongLi commented on PR #137: URL: https://github.com/apache/flink-table-store/pull/137#issuecomment-1153415434
> Thanks for the contribution! It looks good in general. I also left some comments. Beyond that, I have the following questions. > > * For the table `T0`(created via managed table factory), and the table `T1`(created by catalog), what is the difference between them (except for the storage location)? Because I saw `Schema` is recording different options. > * Is there any possibility for a catalog impl (which extends `TableStoreCatalog`) also override `supportsManagedTable`? Thanks for the review, I would like to isolate these two ways, finally, we should just provide catalog way. -- 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]
