[jira] [Commented] (BEAM-1772) Support merging WindowFn other than IntervalWindow on Flink Runner

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on BEAM-1772:
--------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/beam/pull/2331


> Support merging WindowFn other than IntervalWindow on Flink Runner
> ------------------------------------------------------------------
>
>                 Key: BEAM-1772
>                 URL: https://issues.apache.org/jira/browse/BEAM-1772
>             Project: Beam
>          Issue Type: Bug
>          Components: runner-flink
>            Reporter: Ismaël Mejía
>            Assignee: Jingsong Lee
>             Fix For: First stable release
>
>
> Flink currently supports merging IntervalWindows, however if you have a WindowFn who extends IntervalWindow the execution breaks.
> I found this while executing a Pipeline in Flink's batch mode.
> This will involve probably changing the window merging logic in `FlinkMergingReduceFunction.mergeWindows()` and other similar parts to really use the merging logic of the `WindowFn`.



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