[
https://issues.apache.org/jira/browse/HBASE-5601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13249147#comment-13249147
]
[email protected] commented on HBASE-5601:
------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4675/
-----------------------------------------------------------
Review request for hbase.
Summary
-------
HBASE-5601. Add per-column-family data block cache hit ratios. First shot at
the patch.
Changes are as follows (some are backwards incompatible):
- Renamed hbase.RegionServerDynamicStatistics.XXX ->
hbase.regionserver.dyn.XXX
- Renamed the metrics name generation from
hbase.RegionServerDynamicStatistics.tbl.usertable.cf.ycsb.bt.Data.fsBlockReadCacheHitCnt
-> tbl=usertable.cf=ycsb.bt=Data.at=scan.blockCacheHitCount
Notice that tbl.TableName is changed to tbl=TableName, cf.Family is
changed to cf=Family, the fs or compaction prefix is changed to
at=scan or at=compaction. at stands for access type.
- Through using CacheAccessStats/CacheSizeStats, metric names
across per-cf and global are made consistent.
- For compaction, per-block type metrics are disabled. For compaction
we do not need to distinguish between block types.
- Added a configuration "hbase.metrics.dynamic.enabled" for disabling
dynamic metrics. It is enabled by default.
- Changed bloom filter metrics from keyMaybeInBloomCnt -> bloomHitCount
- Removed some code for keeping BC related to META block types
Some changes in HFileReaders are due to automatic trailing whitespace
settings in my eclipse. I can regenerate those parts if it is hard to
review.
This addresses bug HBASE-5601.
https://issues.apache.org/jira/browse/HBASE-5601
Diffs
-----
src/main/java/org/apache/hadoop/hbase/io/hfile/BlockType.java e3c4fe4
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheAccessMetrics.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheAccessStats.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheAccessStatsImpl.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/BlockCache.java 2ba3ab2
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheSizeMetrics.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheSizeStats.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheSizeStatsImpl.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheStats.java ef03075
src/main/java/org/apache/hadoop/hbase/io/hfile/CacheStatsImpl.java
PRE-CREATION
src/main/java/org/apache/hadoop/hbase/io/hfile/DoubleBlockCache.java e952ef9
src/main/java/org/apache/hadoop/hbase/io/hfile/HFileReaderV1.java 79ffee0
src/main/java/org/apache/hadoop/hbase/io/hfile/HFileReaderV2.java aa99add
src/main/java/org/apache/hadoop/hbase/io/hfile/LruBlockCache.java 91562f7
src/main/java/org/apache/hadoop/hbase/io/hfile/SimpleBlockCache.java 87634ba
src/main/java/org/apache/hadoop/hbase/io/hfile/slab/SingleSizeCache.java
1795229
src/main/java/org/apache/hadoop/hbase/io/hfile/slab/SlabCache.java 251ccd4
src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java f1a68e0
src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 8a61f7d
src/main/java/org/apache/hadoop/hbase/regionserver/metrics/RegionServerDynamicMetrics.java
3b46dd8
src/main/java/org/apache/hadoop/hbase/regionserver/metrics/RegionServerMetrics.java
da08532
src/main/java/org/apache/hadoop/hbase/regionserver/metrics/SchemaMetrics.java
5ad7406
src/main/resources/hbase-default.xml 44ee689
src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java f7430ee
src/test/java/org/apache/hadoop/hbase/io/hfile/TestForceCacheImportantBlocks.java
5f8214e
src/test/java/org/apache/hadoop/hbase/io/hfile/TestLruBlockCache.java d980948
src/test/java/org/apache/hadoop/hbase/io/hfile/TestScannerSelectionUsingTTL.java
17284a3
src/test/java/org/apache/hadoop/hbase/regionserver/TestBlocksRead.java
bd83a71
src/test/java/org/apache/hadoop/hbase/regionserver/metrics/TestSchemaMetrics.java
dbe425f
Diff: https://reviews.apache.org/r/4675/diff
Testing
-------
Tested with existing unit tests, and manually using FileContext
Thanks,
enis
> Add per-column-family data block cache hit ratios
> -------------------------------------------------
>
> Key: HBASE-5601
> URL: https://issues.apache.org/jira/browse/HBASE-5601
> Project: HBase
> Issue Type: Improvement
> Reporter: Mikhail Bautin
> Assignee: Enis Soztutar
>
> In addition to the overall block cache hit ratio it would be extremely useful
> to have per-column-family data block cache hit ratio metrics.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
