[FLINK-9299] ProcessWindowFunction documentation Java examples have errors

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.4.2
• Fix Version/s: 1.5.0
• Component/s: Documentation
• Assignee: vinoyang
• Reporter: Kenneth William Krugler
• Created: 04/May/18 22:45
• Updated: 17/May/18 18:27
• Resolved: 17/May/18 18:27

Description

In looking at https://ci.apache.org/projects/flink/flink-docs-release-1.4/dev/stream/operators/windows.html#processwindowfunction-with-incremental-aggregation, I noticed a few errors...

• "This allows to incrementally compute windows" should be "This allows it to incrementally compute windows"
• DataStream<Tuple2<String, Long> input = ...; should be DataStream<Tuple2<String, Long>> input = ...;
• The getResult() method needs to cast one of the accumulator values to a double, if that's what it is going to return.
• MyProcessWindowFunction needs to extend, not implement ProcessWindowFunction
• MyProcessWindowFunction needs to implement a process() method, not an apply() method.
• The call to .timeWindow takes a Time parameter, not a window assigner.

 

Issue Links

• links to: GitHub Pull Request #6000
• links to: GitHub Pull Request #6001