jstorm-runner: Fix duplicated update of metric counter
Project: http://git-wip-us.apache.org/repos/asf/beam/repo Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/240f61bc Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/240f61bc Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/240f61bc Branch: refs/heads/jstorm-runner Commit: 240f61bc6baab9e698b2b6144688853a8371658f Parents: 61e9fa6 Author: basti.lj <[email protected]> Authored: Wed Aug 9 16:03:50 2017 +0800 Committer: Pei He <[email protected]> Committed: Sat Aug 19 12:03:00 2017 +0800 ---------------------------------------------------------------------- .../org/apache/beam/runners/jstorm/translation/MetricsReporter.java | 1 + 1 file changed, 1 insertion(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/beam/blob/240f61bc/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 82d8bdc..5b60b03 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 @@ -74,6 +74,7 @@ class MetricsReporter { AsmCounter counter = metricClient.registerCounter(metricName); Long incValue = (oldValue == null ? updateValue : updateValue - oldValue); counter.update(incValue); + reportedCounters.put(metricName, incValue); } } }
