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

Eshcar Hillel commented on HBASE-15016:
---------------------------------------

Looking at the code I'm trying to understand if using the version numbers in 
the cells is good enough for updating the wal.
>From what I see now it seems that it is; the solution we suggested before was 
>complicated and required getting the wal sequence number from the region, but 
>actually the wal sequence id are stamped in the cell in the HRegion method 
>processRowsWithLocks(), when performing atomic mutation in the region.
I hope I am not missing anything.
Is this the right way to describe it?

So, for your question [~stack], I will remove the getWalSequenceId() method 
from RegionServicesForStores.

HRegion is already implementing 3(!) interfaces. I prefer not to add the 4th 
interface and instead use composition for RegionServicesForStores.
But if you insist I can make it an interface.




> StoreServices facility in Region
> --------------------------------
>
>                 Key: HBASE-15016
>                 URL: https://issues.apache.org/jira/browse/HBASE-15016
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Eshcar Hillel
>            Assignee: Eshcar Hillel
>         Attachments: HBASE-15016-V01.patch, HBASE-15016-V02.patch, 
> HBASE-15016-V03.patch, HBASE-15016-V04.patch, Regioncounters.pdf, 
> suggestion.patch
>
>
> The default implementation of a memstore ensures that between two flushes the 
> memstore size increases monotonically. Supporting new memstores that store 
> data in different formats (specifically, compressed), or that allows to 
> eliminate data redundancies in memory (e.g., via compaction), means that the 
> size of the data stored in memory can decrease even between two flushes. This 
> requires memstores to have access to facilities that manipulate region 
> counters and synchronization.
> This subtasks introduces a new region interface -- StoreServices, through 
> which store components can access these facilities.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to