[
https://issues.apache.org/jira/browse/FLINK-3614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15198121#comment-15198121
]
ASF GitHub Bot commented on FLINK-3614:
---------------------------------------
GitHub user aljoscha opened a pull request:
https://github.com/apache/flink/pull/1805
[FLINK-3614] Remove Non-Keyed Window Operator
Instead we use a dummy KeySelector and the regular WindowOperator now.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/aljoscha/flink
window-remove-non-keyed-operator
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1805.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1805
----
commit 0730c5d46168ef530dbd4058e44f98973628af0e
Author: Aljoscha Krettek <aljoscha.kret...@gmail.com>
Date: 2016-03-16T14:59:05Z
[FLINK-3614] Remove Non-Keyed Window Operator
Instead we use a dummy KeySelector and the regular WindowOperator now.
----
> Remove Non-Keyed Window Operator
> --------------------------------
>
> Key: FLINK-3614
> URL: https://issues.apache.org/jira/browse/FLINK-3614
> Project: Flink
> Issue Type: Improvement
> Components: Streaming
> Reporter: Aljoscha Krettek
> Assignee: Aljoscha Krettek
>
> I propose to remove the special Non-Keyed Window Operator and implement
> non-parallel windows by using the standard WindowOperator with a dummy
> KeySelector.
> Maintaining everything for two WindowOperators is a huge burden. The
> implementation is completely separate by now. For example, the Non-Keyed
> window operator does not use the StateBackend for state, i.e. cannot use
> RocksDB. Also, with upcoming changes (Merging/Session windows, aligned
> windows) this will only increase the maintenance burden.
> Also, the fast AlignedProcessingTimeWindows operators also only support the
> Parallel/Keyed case.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
