[FLINK-5899] Fix the bug in EventTimeTumblingWindow for non-partialMerge aggregate

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: Table SQL / API
• Assignee: Shaoxuan Wang
• Reporter: Shaoxuan Wang
• Created: 24/Feb/17 03:33
• Updated: 24/Feb/17 16:37
• Resolved: 24/Feb/17 16:37

Description

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.

Issue Links

• links to: GitHub Pull Request #3405