[
https://issues.apache.org/jira/browse/FLINK-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14128725#comment-14128725
]
ASF GitHub Bot commented on FLINK-1096:
---------------------------------------
GitHub user sekruse opened a pull request:
https://github.com/apache/incubator-flink/pull/117
[FLINK-1096] Correction to histogram accumulator
* each key is associated with the number of times it was inserted into the
accumulator
* backed histogram with a tree map to present the entries sorted by key
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sekruse/incubator-flink FLINK-1096
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-flink/pull/117.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 #117
----
commit 5080784896b8265bb58bf5c3fb471552c1277b37
Author: Sebastian Kruse <[email protected]>
Date: 2014-09-10T16:59:09Z
[FLINK-1096] Correction to histogram accumulator
* each key is associated with the number of times it was inserted into the
accumulator
* backed histogram with a tree map to present the entries sorted by key
----
> Unexpected Histogram accumulator behavior
> -----------------------------------------
>
> Key: FLINK-1096
> URL: https://issues.apache.org/jira/browse/FLINK-1096
> Project: Flink
> Issue Type: Bug
> Reporter: Sebastian Kruse
> Assignee: Sebastian Kruse
> Priority: Trivial
>
> The {{Histogram}} accumulator uses the given value as both key and value for
> the histogram map. Hence, the histogram always contains entries like {{key ->
> n * key}} where {{n}} is the number of times that {{key}} has been added to
> the histogram.
> Supposedly, adding {{key}} into the histogram {{n}} times should create the
> entr {{key -> n}} instead.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
