[ https://issues.apache.org/jira/browse/FLINK-7446?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16179069#comment-16179069 ]
ASF GitHub Bot commented on FLINK-7446: --------------------------------------- Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/4710#discussion_r140777209 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/schema/StreamTableSourceTable.scala --- @@ -53,18 +53,11 @@ object StreamTableSourceTable { val original = TableEnvironment.getFieldIndices(tableSource) - // append rowtime marker - val withRowtime = if (rowtime.isDefined) { --- End diff -- remove unused `rowtime` > Support to define an existing field as the rowtime field for TableSource > ------------------------------------------------------------------------ > > Key: FLINK-7446 > URL: https://issues.apache.org/jira/browse/FLINK-7446 > Project: Flink > Issue Type: Improvement > Components: Table API & SQL > Reporter: Jark Wu > Assignee: Fabian Hueske > > Currently {{DefinedRowtimeAttribute}} can define an appended rowtime field > for a {{TableSource}}. But it would be helpful if we can support to define an > existing field as the rowtime field. Just like registering a DataStream, the > rowtime field can be appended but also can replace an existing field. -- This message was sent by Atlassian JIRA (v6.4.14#64029)