[
https://issues.apache.org/jira/browse/HBASE-10648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13934833#comment-13934833
]
Yu Li commented on HBASE-10648:
-------------------------------
{quote}
Sorry I will need some more time to post a patch for HBASE-10713
{quote}
I see, it's ok, thanks for the update. In this case I might start writing some
code by myself (you know, the deadline thing :-)), and I'll appreciate a lot if
you can send your drafted patch to me through email, before it's ready for
public. Just want to avoid duplicated work as much as possible.
{quote}
BTW I am not sure whether this kind of a change can go into an older version
like 94
{quote}
Sorry but what's preventing the change to go into 94? The CSLM issue (when
there are too many KVs in it) also exists in v94, and we also want to get
relieved from that:-) Of course I'm not expecting the whole HBASE-10191
solution go into 94, but after making Memstore impl pluggable, at least
in-memory flush impl would fit well in 0.94, right?
{quote}
You want to internally maintain that?
{quote}
It's not an easy decision to upgrade to a new version in product env, and I'm
afraid we'll stay on 94 for some while, so yes we will maintain this
internally. But of course I'll try to contribute back to 94 branch after fully
test, so others using 94 can also benefit.
> Pluggable Memstore
> ------------------
>
> Key: HBASE-10648
> URL: https://issues.apache.org/jira/browse/HBASE-10648
> Project: HBase
> Issue Type: Sub-task
> Reporter: Anoop Sam John
> Assignee: Anoop Sam John
> Attachments: HBASE-10648-0.94_v1.patch, HBASE-10648.patch,
> HBASE-10648_V2.patch, HBASE-10648_V3.patch, HBASE-10648_V4.patch,
> HBASE-10648_V5.patch, HBASE-10648_V6.patch
>
>
> Make Memstore into an interface implementation. Also make it pluggable by
> configuring the FQCN of the impl.
> This will allow us to have different impl and optimizations in the Memstore
> DataStructure and the upper layers untouched.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
