dianfu commented on issue #7626: [FLINK-11450][table] Port and move TableSource and TableSink to flink-table-common URL: https://github.com/apache/flink/pull/7626#issuecomment-459732168 @twalthr Thanks for the suggestion. Make sense to me. Regarding to StreamableDescriptor, I'd like to declare it an interface and doesn't extend TableDescriptor anymore. The reason is that TableDescriptor is an abstract class and StreamableDescriptor has to be declared as an abstract class if it extends TableDescriptor. Given that StreamTableDescriptor extends both ConnectTableDescriptor and StreamableDescriptor, it will be impossible. What's your thought about this change?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
