[FLINK-6257] Post-pass OVER windows

• Type: Sub-task
• Status: Closed
• Priority: Critical
• Resolution: Done
• Affects Version/s: 1.3.0
• Fix Version/s: 1.3.0
• Component/s: Table SQL / API
• Assignee: sunjincheng
• Reporter: Fabian Hueske
• Created: 03/Apr/17 16:17
• Updated: 07/May/17 13:52
• Resolved: 07/May/17 13:52

Description

The OVER windows have been implemented by several contributors.
We should do a post pass over the contributed code and improve a few things.

• Functionality
  • Currently every time attribute is allowed as ORDER BY attribute. We must check that this is actually a time indicator (procTime(), rowTime()) and that the order is ASCENDING.
• Documentation
  • Add documentation for OVER windows
• Code style
  • Consistent naming of ProcessFunctions and methods
• Tests
  • Move the OVER window tests out of SqlITCase into a dedicated class
  • Move the OVER window tests out of WindowAggregateTest into a dedicated class
  • Add tests based on the test harness for all ProcessFunctions similar to BoundedProcessingOverRangeProcessFunction. The tests should include exact boundary checks for range windows and check for proper parallelization with multiple keys.

Issue Links

• links to: GitHub Pull Request #3681
• links to: GitHub Pull Request #3697