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:11:07 UTC

svn commit: r1181465 - /hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/PersistentMetricsTimeVaryingRate.java

Author: nspiegelberg
Date: Tue Oct 11 02:11:07 2011
New Revision: 1181465

URL: http://svn.apache.org/viewvc?rev=1181465&view=rev
Log:
Fix small div-by-zero error that Karthik found

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=1181465&r1=1181464&r2=1181465&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:11:07 2011
@@ -128,7 +128,7 @@ public class PersistentMetricsTimeVaryin
     if (curOps > 0) {
       long curTime = this.getPreviousIntervalAverageTime();
       long totalTime = curTime * curOps;
-      if (totalTime / curTime == curOps) {
+      if (curTime == 0 || totalTime / curTime == curOps) {
         super.inc(curOps, totalTime);
       } else {
         LOG.info("Stats for " + this.getName() + " overflowed! resetting");