Github user twalthr commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6201#discussion_r198146395
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/connector/TableConnectorFactoryService.scala
 ---
    @@ -141,22 +143,26 @@ object TableSourceFactoryService extends Logging {
     
         // check for supported properties
         plainProperties.foreach { k =>
    -      if (!supportedProperties.contains(k)) {
    +      if (!k.equals(TableDescriptorValidator.TABLE_TYPE) && 
!supportedProperties.contains(k)) {
    --- End diff --
    
    The table type must not be part of the supported properties as it is 
defined in a separate method in the `TableConnectorFactory` interface.


---

Reply via email to