[ 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)