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

    https://github.com/apache/flink/pull/4894#discussion_r147339574
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/catalog/ExternalTableSourceUtil.scala
 ---
    @@ -124,9 +124,15 @@ object ExternalTableSourceUtil extends Logging {
               } else {
                 FlinkStatistic.UNKNOWN
               }
    +
               convertedTableSource match {
    -            case s : StreamTableSource[_] => new StreamTableSourceTable(s, 
flinkStatistic)
    -            case _ => new TableSourceTable(convertedTableSource, 
flinkStatistic)
    +            case s: StreamTableSource[_] =>
    --- End diff --
    
    In that case a `StreamTableSourceTable` is registered. I kept the previous 
behavior. 
    We can fix it in a follow up issue and make the decision based on the 
execution environment.


---

Reply via email to