Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Wed, 20 Dec 2017 03:56:00 +0000 (UTC)
From: "Guangxu Cheng (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.13126150.1513742140000.499465.1513742160087@Atlassian.JIRA>
In-Reply-To: <JIRA.13126150.1513742140000@Atlassian.JIRA>
References: <JIRA.13126150.1513742140000@Atlassian.JIRA> <JIRA.13126150.1513742140901@jira-lw-us.apache.org>
Subject: [jira] [Created] (HBASE-19561) maxCacheSize in CacheEvictionStats
 can't be accumulated repeatedly When dealing with each region
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Wed, 20 Dec 2017 03:56:10 -0000

Guangxu Cheng created HBASE-19561:
-------------------------------------

             Summary: maxCacheSize in CacheEvictionStats can't be accumulated repeatedly When dealing with each region
                 Key: HBASE-19561
                 URL: https://issues.apache.org/jira/browse/HBASE-19561
             Project: HBase
          Issue Type: Bug
    Affects Versions: 2.0.0
            Reporter: Guangxu Cheng
            Assignee: Guangxu Cheng


{code:title=HRegionServer.java|borderStyle=solid}
  public CacheEvictionStats clearRegionBlockCache(Region region) {
    BlockCache blockCache = this.getCacheConfig().getBlockCache();
    long evictedBlocks = 0;

    for(Store store : region.getStores()) {
      for(StoreFile hFile : store.getStorefiles()) {
        evictedBlocks += blockCache.evictBlocksByHfileName(hFile.getPath().getName());
      }
    }

    return CacheEvictionStats.builder()
        .withEvictedBlocks(evictedBlocks)
        .withMaxCacheSize(blockCache.getMaxSize())
        .build();
  }
{code}

BlockCache is the regionserver level object.If we accumulate this value for each region, we will get a wrong result.
The value of maxCacheSize should be set to BlockCache.getMaxSize() after processing all the regions on this regionserver.


--
This message was sent by Atlassian JIRA
(v6.4.14#64029)