You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by pe...@apache.org on 2017/08/20 15:03:45 UTC
[45/53] [abbrv] beam git commit: jstorm-runner: Fix duplicated update
of metric counter
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 <ba...@alibaba-inc.com>
Authored: Wed Aug 9 16:03:50 2017 +0800
Committer: Pei He <pe...@apache.org>
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);
}
}
}