[
https://issues.apache.org/jira/browse/HBASE-8024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13608077#comment-13608077
]
Sergey Shelukhin commented on HBASE-8024:
-----------------------------------------
DefaultStoreFlusher seems to be missing from the patch.
Need to see it to comment more :)
So far:
bq. // Add new file to store files. Clear snapshot too while we have
bq. // the Store write lock.
There's no lock... probably needs documentation if it has to be held?
bq. + super(cacheFlushId, conf, store);
bq. + }
nit: whitespace, elsewhere too
> Make Store flush algorithm pluggable
> ------------------------------------
>
> Key: HBASE-8024
> URL: https://issues.apache.org/jira/browse/HBASE-8024
> Project: HBase
> Issue Type: Sub-task
> Components: regionserver
> Affects Versions: 0.95.0, 0.96.0, 0.94.5
> Reporter: Maryann Xue
> Attachments: HBASE-8024-trunk.patch
>
>
> The idea is to make "StoreFlusher" an interface instead of an implementation
> class, and have the original StoreFlusher as the default store flush impl.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
