You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 04:08:07 UTC
svn commit: r1181434 -
/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java
Author: nspiegelberg
Date: Tue Oct 11 02:08:07 2011
New Revision: 1181434
URL: http://svn.apache.org/viewvc?rev=1181434&view=rev
Log:
Fixed minor bug where metrics weren't persisting after the first reset.
Modified:
hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java
Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java?rev=1181434&r1=1181433&r2=1181434&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java Tue Oct 11 02:08:07 2011
@@ -82,6 +82,7 @@ public class PersistentMetricsTimeVaryin
if (reset) {
// use the previous avg as our starting min/max/avg
super.inc(getPreviousIntervalAverageTime());
+ reset = false;
} else {
// maintain the stats that pushMetric() cleared
maintainStats();