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


---

Reply via email to