Github user fhueske commented on the issue: https://github.com/apache/flink/pull/5240 Hi Timo, the PR looks good overall. I've made a few suggestion mostly about renaming methods or extending docs. I'd also propose to add a `supportedProperties()` method to `TableSourceFactory` that can be used to validate whether the factory supports all user-provided properties of a connector or format. What do you think? Fabian
---