jstorm-runner: Fix incorrect updating of counter metrics
Project: http://git-wip-us.apache.org/repos/asf/beam/repo Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/c9526869 Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/c9526869 Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/c9526869 Branch: refs/heads/jstorm-runner Commit: c95268691f78a629866f722df1a3f7ef5e76a256 Parents: 557d703 Author: basti.lj <basti...@alibaba-inc.com> Authored: Wed Aug 30 10:45:45 2017 +0800 Committer: Pei He <p...@apache.org> Committed: Wed Aug 30 14:55:17 2017 +0800 ---------------------------------------------------------------------- .../apache/beam/runners/jstorm/translation/MetricsReporter.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/beam/blob/c9526869/runners/jstorm/src/main/java/org/apache/beam/runners/jstorm/translation/MetricsReporter.java ---------------------------------------------------------------------- diff --git a/runners/jstorm/src/main/java/org/apache/beam/runners/jstorm/translation/MetricsReporter.java b/runners/jstorm/src/main/java/org/apache/beam/runners/jstorm/translation/MetricsReporter.java index e7f3285..0315a59 100644 --- a/runners/jstorm/src/main/java/org/apache/beam/runners/jstorm/translation/MetricsReporter.java +++ b/runners/jstorm/src/main/java/org/apache/beam/runners/jstorm/translation/MetricsReporter.java @@ -72,7 +72,7 @@ class MetricsReporter { AsmCounter counter = metricClient.registerCounter(metricName); Long incValue = (oldValue == null ? updateValue : updateValue - oldValue); counter.update(incValue); - reportedCounters.put(metricName, incValue); + reportedCounters.put(metricName, updateValue); } } }