You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2018/12/14 00:15:13 UTC

[45/50] [abbrv] hbase git commit: HBASE-21212 Wrong flush time when update flush metric

HBASE-21212 Wrong flush time when update flush metric


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/a4baeebd
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/a4baeebd
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/a4baeebd

Branch: refs/heads/branch-1.3
Commit: a4baeebdfc16878011633aa548b90dd53afddc37
Parents: 09069df
Author: Allan Yang <al...@apache.org>
Authored: Wed Sep 26 19:40:43 2018 +0800
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Dec 12 19:25:46 2018 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/regionserver/HRegion.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/a4baeebd/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
index d41e7cb..71785fd 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
@@ -2639,7 +2639,7 @@ public class HRegion implements HeapSize, PropagatingConfigurationObserver, Regi
     status.setStatus(msg);
 
     if (rsServices != null && rsServices.getMetrics() != null) {
-      rsServices.getMetrics().updateFlush(time - startTime,
+      rsServices.getMetrics().updateFlush(time,
         totalFlushableSizeOfFlushableStores, flushedOutputFileSize);
     }