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

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

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

    https://github.com/apache/flink/pull/3266#discussion_r100001509
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/AggregateUtil.scala
 ---
    @@ -210,14 +210,14 @@ object AggregateUtil {
             else {
               // for non-incremental aggregations
               new DataSetTumbleTimeWindowAggReduceGroupFunction(
    -            intermediateRowArity - 1,
    +            intermediateRowArity,
                 asLong(size),
                 startPos,
                 endPos,
                 aggregates,
                 groupingOffsetMapping,
                 aggOffsetMapping,
    -            intermediateRowArity,
    +            intermediateRowArity + 1, // the additional field is used to 
store the time attribute
    --- End diff --
    
    good catch !


> Add non-grouped session windows for batch tables
> ------------------------------------------------
>
>                 Key: FLINK-5219
>                 URL: https://issues.apache.org/jira/browse/FLINK-5219
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API & SQL
>            Reporter: sunjincheng
>            Assignee: sunjincheng
>
> Add non-grouped session windows for batch tables as described in 
> [FLIP-11|https://cwiki.apache.org/confluence/display/FLINK/FLIP-11%3A+Table+API+Stream+Aggregations].



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to