flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-6584) Support multiple consecutive windows in SQL
Date Tue, 27 Jun 2017 17:45:01 GMT

    [ https://issues.apache.org/jira/browse/FLINK-6584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16065188#comment-16065188
] 

ASF GitHub Bot commented on FLINK-6584:
---------------------------------------

Github user sunjincheng121 commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4199#discussion_r124329598
  
    --- Diff: flink-libraries/flink-table/src/main/java/org/apache/calcite/sql/fun/SqlStdOperatorTable.java
---
    @@ -1900,7 +1900,7 @@ public void unparse(
     		new SqlGroupFunction(SqlKind.TUMBLE, null,
     			OperandTypes.or(OperandTypes.DATETIME_INTERVAL,
     				OperandTypes.DATETIME_INTERVAL_TIME)) {
    -			@Override List<SqlGroupFunction> getAuxiliaryFunctions() {
    +			@Override public List<SqlGroupFunction> getAuxiliaryFunctions() {
    --- End diff --
    
    Dose this method only using in `SqlStdOperatorTable.java`? if so , why we add `public`.
I find calcite have not add this modifier.


> Support multiple consecutive windows in SQL
> -------------------------------------------
>
>                 Key: FLINK-6584
>                 URL: https://issues.apache.org/jira/browse/FLINK-6584
>             Project: Flink
>          Issue Type: New Feature
>          Components: Table API & SQL
>            Reporter: Timo Walther
>            Assignee: Timo Walther
>
> Right now, the Table API supports multiple consecutive windows as follows:
> {code}
> val table = stream.toTable(tEnv, 'rowtime.rowtime, 'int, 'double, 'float, 'bigdec, 'string)
> val t = table
>   .window(Tumble over 2.millis on 'rowtime as 'w)
>   .groupBy('w)
>   .select('w.rowtime as 'rowtime, 'int.count as 'int)
>   .window(Tumble over 4.millis on 'rowtime as 'w2)
>   .groupBy('w2)
>   .select('w2.rowtime, 'w2.end, 'int.count)
> {code}
> Similar behavior should be supported by the SQL API as well. We need to introduce a new
auxiliary group function, but this should happen in sync with Apache Calcite.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message