You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2015/08/05 02:20:27 UTC

hbase git commit: HBASE-14185 Incorrect region names logged by MemStoreFlusher (Biju Nair)

Repository: hbase
Updated Branches:
  refs/heads/master 1b0b67fb7 -> a0d72051d


HBASE-14185 Incorrect region names logged by MemStoreFlusher (Biju Nair)


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

Branch: refs/heads/master
Commit: a0d72051dbace9dc4ec6ab288f2f6553e2ee7307
Parents: 1b0b67f
Author: tedyu <yu...@gmail.com>
Authored: Tue Aug 4 17:20:23 2015 -0700
Committer: tedyu <yu...@gmail.com>
Committed: Tue Aug 4 17:20:23 2015 -0700

----------------------------------------------------------------------
 .../org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/a0d72051/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
index 644e89b..40c5046 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
@@ -197,12 +197,12 @@ class MemStoreFlusher implements FlushRequester {
            ServerRegionReplicaUtil.isRegionReplicaStoreFileRefreshEnabled(conf) &&
            (bestRegionReplica.getMemstoreSize()
                > secondaryMultiplier * regionToFlush.getMemstoreSize()))) {
-        LOG.info("Refreshing storefiles of region " + regionToFlush +
+        LOG.info("Refreshing storefiles of region " + bestRegionReplica +
           " due to global heap pressure. memstore size=" + StringUtils.humanReadableInt(
             server.getRegionServerAccounting().getGlobalMemstoreSize()));
         flushedOne = refreshStoreFilesAndReclaimMemory(bestRegionReplica);
         if (!flushedOne) {
-          LOG.info("Excluding secondary region " + regionToFlush +
+          LOG.info("Excluding secondary region " + bestRegionReplica +
               " - trying to find a different region to refresh files.");
           excludedRegions.add(bestRegionReplica);
         }