[
https://issues.apache.org/jira/browse/FLINK-6780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16031012#comment-16031012
]
ASF GitHub Bot commented on FLINK-6780:
---------------------------------------
Github user twalthr commented on the issue:
https://github.com/apache/flink/pull/4023
Thanks @haohui. The change looks good. I will merge this.
> ExternalTableSource should add time attributes in the row type
> --------------------------------------------------------------
>
> Key: FLINK-6780
> URL: https://issues.apache.org/jira/browse/FLINK-6780
> Project: Flink
> Issue Type: Bug
> Components: Table API & SQL
> Reporter: Haohui Mai
> Assignee: Haohui Mai
> Priority: Critical
>
> We observed that all streaming queries that refer to external tables fail
> when the Volcano planner converting {{LogicalTableScan}} to
> {{FlinkLogicalTableSourceScan}}:
> {noformat}
> Type mismatch:
> rowtype of new rel:
> RecordType(<table schema>, TIMESTAMP(3) NOT NULL proctime) NOT NULL
> rowtype of set:
> RecordType(<table schema>, ...) NOT NULL
> {noformat}
> Tables that are registered through
> {{StreamTableEnvironment#registerTableSource()}} do not suffer from this
> problem as {{StreamTableSourceTable}} adds the processing time / event time
> attribute automatically.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
