[ https://issues.apache.org/jira/browse/HBASE-4482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13121735#comment-13121735 ]
jirapos...@reviews.apache.org commented on HBASE-4482: ------------------------------------------------------ bq. On 2011-10-06 05:30:17, Ted Yu wrote: bq. > src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java, line 397 bq. > <https://reviews.apache.org/r/2051/diff/5/?file=48444#file48444line397> bq. > bq. > Should be (offHeapCacheSize <= 0 || !enableOffHeapCache) Fixed. bq. On 2011-10-06 05:30:17, Ted Yu wrote: bq. > src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java, line 401 bq. > <https://reviews.apache.org/r/2051/diff/5/?file=48444#file48444line401> bq. > bq. > Whitespace. Fixed. bq. On 2011-10-06 05:30:17, Ted Yu wrote: bq. > src/main/java/org/apache/hadoop/hbase/io/hfile/slab/SlabCache.java, line 225 bq. > <https://reviews.apache.org/r/2051/diff/5/?file=48442#file48442line225> bq. > bq. > Indent to the right two spaces. Fixed. - Li ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2051/#review2384 ----------------------------------------------------------- On 2011-10-06 04:25:43, Li Pi wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/2051/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-10-06 04:25:43) bq. bq. bq. Review request for hbase. bq. bq. bq. Summary bq. ------- bq. bq. Bugfix, kills a race condition. bq. bq. Ignore r1, thats the wrong patch. bq. bq. bq. This addresses bug HBASE-4482. bq. https://issues.apache.org/jira/browse/HBASE-4482 bq. bq. bq. Diffs bq. ----- bq. bq. src/main/java/org/apache/hadoop/hbase/io/hfile/slab/SingleSizeCache.java 3798a06 bq. src/main/java/org/apache/hadoop/hbase/io/hfile/slab/SlabCache.java fe8b95a bq. src/main/java/org/apache/hadoop/hbase/io/hfile/slab/SlabItemEvictionWatcher.java 91b1603 bq. src/main/java/org/apache/hadoop/hbase/regionserver/StoreFile.java 4072387 bq. src/test/java/org/apache/hadoop/hbase/io/hfile/CacheTestUtils.java 0814f41 bq. src/test/java/org/apache/hadoop/hbase/io/hfile/slab/TestSingleSizeCache.java e021780 bq. src/test/java/org/apache/hadoop/hbase/io/hfile/slab/TestSlabCache.java 8dd5159 bq. bq. Diff: https://reviews.apache.org/r/2051/diff bq. bq. bq. Testing bq. ------- bq. bq. Looped tests. bq. bq. bq. Thanks, bq. bq. Li bq. bq. > Race Condition Concerning Eviction in SlabCache > ----------------------------------------------- > > Key: HBASE-4482 > URL: https://issues.apache.org/jira/browse/HBASE-4482 > Project: HBase > Issue Type: Sub-task > Reporter: Li Pi > Assignee: Li Pi > Priority: Blocker > Fix For: 0.92.0 > > Attachments: hbase-4482v1.txt, hbase-4482v2.txt > > -- 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