GitHub user F30 opened a pull request:
https://github.com/apache/storm/pull/2362
Improve mention of aggregate() in the partitionAggregate() docs
This is a revival of #1018: It took a while, but I have now rebased the
changes onto the current master and adjusted the base branch.
However, #1018 is in "missing common history" mode after force-pushing the
new commits. Changing the base branch didn't cause the diff to be recomputed,
so the Pull Request cannot be re-opened.
Therefore, this new PR is required.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/F30/storm partitionAggregate-docs
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2362.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 #2362
----
commit 9710434bb12d96d20e00d41b85e520f4192d3fde
Author: Felix Dreissig <[email protected]>
Date: 2016-01-15T23:48:12Z
Improve mention of aggregate() in the partitionAggregate() docs
It was not obvious (to me) that aggregate() is covered in detail at
another place in the document. That specific section describes the
details of the optimization much more exhaustively.
Also, it was unclear what the benefits should be compared to, because
other aggregator types are only introduced afterwards.
----
---
