[
https://issues.apache.org/jira/browse/FLINK-5884?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996313#comment-15996313
]
ASF GitHub Bot commented on FLINK-5884:
---------------------------------------
Github user sunjincheng121 commented on a diff in the pull request:
https://github.com/apache/flink/pull/3808#discussion_r114697863
--- Diff:
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/datastream/DataStreamAggregateITCase.scala
---
@@ -62,10 +62,10 @@ class DataStreamAggregateITCase extends
StreamingMultipleProgramsTestBase {
val stream = env
.fromCollection(data)
.assignTimestampsAndWatermarks(new TimestampWithEqualWatermark())
- val table = stream.toTable(tEnv, 'long, 'int, 'double, 'float,
'bigdec, 'string)
+ val table = stream.toTable(tEnv, 'long.rowtime, 'int, 'double, 'float,
'bigdec, 'string)
--- End diff --
I think we can using `.rowtime` in this way as follwow:
`stream.toTable(tEnv, 'long, 'int, 'double, 'float, 'bigdec, 'string,
'rowtime.rowtime)`
Because we do not need the value of `'long` field. It's just a indicators
in stream.
> Integrate time indicators for Table API & SQL
> ---------------------------------------------
>
> Key: FLINK-5884
> URL: https://issues.apache.org/jira/browse/FLINK-5884
> Project: Flink
> Issue Type: New Feature
> Components: Table API & SQL
> Reporter: Timo Walther
> Assignee: Timo Walther
> Priority: Blocker
> Fix For: 1.3.0
>
>
> We already discussed the need for a proper integration of time indicators
> (event-time or processing-time) for both the Table API & SQL on the ML:
> http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Table-API-SQL-indicators-for-event-and-processing-time-tp15927.html
> This issue will track the progress. I will work on a design document how we
> can solve this issue.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
