flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephan Ewen (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (FLINK-9299) ProcessWindowFunction documentation Java examples have errors
Date Thu, 17 May 2018 18:28:00 GMT

     [ https://issues.apache.org/jira/browse/FLINK-9299?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Stephan Ewen resolved FLINK-9299.
---------------------------------
       Resolution: Fixed
    Fix Version/s: 1.5.0

Fixed in
  - 1.5.0 via 94391d89a49abda9e1b82cc2bc44daa8159da9e1dd6
  - 1.6.0 via 22531a87158a725804734b27e091ff6597686

> ProcessWindowFunction documentation Java examples have errors
> -------------------------------------------------------------
>
>                 Key: FLINK-9299
>                 URL: https://issues.apache.org/jira/browse/FLINK-9299
>             Project: Flink
>          Issue Type: Bug
>          Components: Documentation
>    Affects Versions: 1.4.2
>            Reporter: Ken Krugler
>            Assignee: vinoyang
>            Priority: Minor
>             Fix For: 1.5.0
>
>
> 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.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message