You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Jark Wu (Jira)" <ji...@apache.org> on 2020/04/02 11:32:00 UTC
[jira] [Commented] (FLINK-16160) Schema#proctime and Schema#rowtime
don't work in TableEnvironment#connect code path
[ https://issues.apache.org/jira/browse/FLINK-16160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17073643#comment-17073643 ]
Jark Wu commented on FLINK-16160:
---------------------------------
Hi [~docete], I think this PR mixes the bug fix and new feature. Could you separate them into two issues?
The first issue is just fix proctime() doesn't work for {{createTemporaryTable}}, and I think the root cause is [this line|https://github.com/apache/flink/blob/release-1.10/flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/descriptors/ConnectTableDescriptor.java#L143]. We should fix this for 1.10 too.
The other issue is introduce a new API to define rowtime and proctime.
> Schema#proctime and Schema#rowtime don't work in TableEnvironment#connect code path
> -----------------------------------------------------------------------------------
>
> Key: FLINK-16160
> URL: https://issues.apache.org/jira/browse/FLINK-16160
> Project: Flink
> Issue Type: Sub-task
> Components: Table SQL / API
> Reporter: Zhenghua Gao
> Priority: Critical
> Labels: pull-request-available
> Fix For: 1.11.0
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> In ConnectTableDescriptor#createTemporaryTable, the proctime/rowtime properties are ignored so the generated catalog table is not correct. We should fix this to let TableEnvironment#connect() support watermark.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)