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

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

Github user fhueske commented on the issue:

    https://github.com/apache/flink/pull/3405
  
    Thanks for the fix @shaoxuan-wang!
    Merging


> Fix the bug in EventTimeTumblingWindow for non-partialMerge aggregate
> ---------------------------------------------------------------------
>
>                 Key: FLINK-5899
>                 URL: https://issues.apache.org/jira/browse/FLINK-5899
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>            Reporter: Shaoxuan Wang
>            Assignee: Shaoxuan Wang
>
> The row length used to initialize 
> DataSetTumbleTimeWindowAggReduceGroupFunction was not set properly. (I think 
> this is introduced by mistake when merging the code).
> We currently lack the built-in non-partial-merge Aggregates. Therefore this 
> has not been captured by the unit test. 
> Reproduce step:
> 1. set the "supportPartial" to false for SumAggregate
> 2. Then both testAllEventTimeTumblingWindowOverTime and 
> testEventTimeTumblingGroupWindowOverTime will fail.



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

Reply via email to