[ https://issues.apache.org/jira/browse/FLINK-4691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15548965#comment-15548965 ]
ASF GitHub Bot commented on FLINK-4691: --------------------------------------- Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/2562#discussion_r81857880 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/api/scala/table/expressionDsl.scala --- @@ -325,33 +337,42 @@ trait ImplicitExpressionOperations { */ def day = toMilliInterval(expr, MILLIS_PER_DAY) - /** + /** * Creates an interval of the given number of hours. * * @return interval of milliseconds */ def hour = toMilliInterval(expr, MILLIS_PER_HOUR) - /** + /** * Creates an interval of the given number of minutes. * * @return interval of milliseconds */ def minute = toMilliInterval(expr, MILLIS_PER_MINUTE) - /** + /** * Creates an interval of the given number of seconds. * * @return interval of milliseconds */ def second = toMilliInterval(expr, MILLIS_PER_SECOND) - /** + /** * Creates an interval of the given number of milliseconds. * * @return interval of milliseconds */ def milli = toMilliInterval(expr, 1) + + // row type + + /** + * Creates a number defining an amount of rows. + * + * @return number of rows + */ + def rows = expr --- End diff -- Doesn't this allow to use `Int.row` wherever `Int` is allowed? Should we return a `RowInterval` expression? > Add group-windows for streaming tables > --------------------------------------- > > Key: FLINK-4691 > URL: https://issues.apache.org/jira/browse/FLINK-4691 > Project: Flink > Issue Type: Sub-task > Components: Table API & SQL > Reporter: Timo Walther > Assignee: Timo Walther > > Add Tumble, Slide, Session group-windows for streaming tables as described in > [FLIP-11|https://cwiki.apache.org/confluence/display/FLINK/FLIP-11%3A+Table+API+Stream+Aggregations]. > > Implementation of group-windows on streaming tables. This includes > implementing the API of group-windows, the logical validation for > group-windows, and the definition of the “rowtime” and “systemtime” keywords. > Group-windows on batch tables won’t be initially supported and will throw an > exception. -- This message was sent by Atlassian JIRA (v6.3.4#6332)