[
https://issues.apache.org/jira/browse/HBASE-5230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191488#comment-13191488
]
Phabricator commented on HBASE-5230:
------------------------------------
nspiegelberg has commented on the revision "[jira] [HBASE-5230] Extend
TestCacheOnWrite to ensure we don't cache data blocks on compaction".
INLINE COMMENTS
src/main/java/org/apache/hadoop/hbase/regionserver/Store.java:764-766
currently, there is no intelligence to estimate the resulting compacted
filesize and cache compactions up to a max size, correct?
src/main/java/org/apache/hadoop/hbase/regionserver/metrics/SchemaMetrics.java:879-880
use this static function to write a toString method?
src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java:4502
this comment should be changed
// read the row, this should be a cache miss because we don't cache on
compaction
src/test/java/org/apache/hadoop/hbase/io/hfile/TestCacheOnWrite.java:254 add
// TODO: need to change this test if we add a cache size threshold for
compactions
src/test/java/org/apache/hadoop/hbase/io/hfile/TestCacheOnWrite.java:295
Assert.assertNull() is nice for clarity
REVISION DETAIL
https://reviews.facebook.net/D1353
> Unit test to ensure compactions don't cache data on write
> ---------------------------------------------------------
>
> Key: HBASE-5230
> URL: https://issues.apache.org/jira/browse/HBASE-5230
> Project: HBase
> Issue Type: Test
> Reporter: Mikhail Bautin
> Assignee: Mikhail Bautin
> Priority: Minor
> Attachments: D1353.1.patch, D1353.2.patch, D1353.3.patch,
> Don-t-cache-data-blocks-on-compaction-2012-01-21_00_53_54.patch,
> Don-t-cache-data-blocks-on-compaction-2012-01-23_10_23_45.patch
>
>
> Create a unit test for HBASE-3976 (making sure we don't cache data blocks on
> write during compactions even if cache-on-write is enabled generally
> enabled). This is because we have very different implementations of
> HBASE-3976 without HBASE-4422 CacheConfig (on top of 89-fb, created by Liyin)
> and with CacheConfig (presumably it's there but not sure if it even works,
> since the patch in HBASE-3976 may not have been committed). We need to create
> a unit test to verify that we don't cache data blocks on write during
> compactions, and resolve HBASE-3976 so that this new unit test does not fail.
--
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
