bowenli86 commented on issue #9172: [FLINK-13313][table] create 
CatalogTableBuilder to support building CatalogTable from descriptors
URL: https://github.com/apache/flink/pull/9172#issuecomment-513370872
 
 
   re: 2, 3 > good idea. Will add
   re: 1 > I'm concerned about renaming since we don't have 
`GenericCatalogTable` and thus `GenericCatalogTableBuilder` would confuse 
users. To most Flink users who are used to original way to create Flink 
source/sink with descriptors, they don't need to understand difference of 
whether a catalog table is generic or not. They just need an equivalent of 
`ExternalCatalogTableBuilder`.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to