hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-14921) Memory optimizations
Date Sat, 16 Apr 2016 03:50:25 GMT

    [ https://issues.apache.org/jira/browse/HBASE-14921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15243995#comment-15243995

stack commented on HBASE-14921:

On patch....

This is still in here:

	 * CellArrayMap is a simple array of Cells and can be allocated only using JVM.

How can something called CellArrayMap be an array of Cells? Whats 'Map' doing in there? It
implements Map, so be nicer to Maps in your description ? (smile)

 And 'only using JVM'... what else? You mean 'on heap'?

We pass comparator on construction and in createSubCellFlatMap? We need it in both places?
The cell array is ordered? Why we need a comparator if it already ordered?

When getCell, we should check the boundaries... the min and max.

It is odd that CellFlatMap implements ConcurrentNavigableMap but...

37	 * The CellFlatMap uses no synchronization primitives, it is assumed to be created by a
38	 * single thread and then it can be read-only by multiple threads.

The parent ImmutableSegment has to know its subclasses? The enums?

Can this be a CellScanner?

	public class MemStoreCompactorIterator implements Iterator<Cell> {

Will be back with closer review... but looks good.

> Memory optimizations
> --------------------
>                 Key: HBASE-14921
>                 URL: https://issues.apache.org/jira/browse/HBASE-14921
>             Project: HBase
>          Issue Type: Sub-task
>    Affects Versions: 2.0.0
>            Reporter: Eshcar Hillel
>            Assignee: Anastasia Braginsky
>         Attachments: CellBlocksSegmentInMemStore.pdf, CellBlocksSegmentinthecontextofMemStore(1).pdf,
HBASE-14921-V01.patch, HBASE-14921-V02.patch, HBASE-14921-V03.patch, IntroductiontoNewFlatandCompactMemStore.pdf
> Memory optimizations including compressed format representation and offheap allocations

This message was sent by Atlassian JIRA

View raw message