hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ramkrishna.s.vasudevan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-11920) Add CP hooks for ReplicationEndPoint
Date Tue, 23 Sep 2014 03:43:34 GMT

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

ramkrishna.s.vasudevan commented on HBASE-11920:
------------------------------------------------

Actually if you see the javadoc for postScannerOpen 
{code}
  * @return an RegionScanner instance to use instead of the base scanner if
   * overriding default behavior, null otherwise
{code}
in preStoreScannerOpen
{code}
a KeyValueScanner instance to use or {@code null} to use the default implementation
{code}
So the caller is handling this, if not null then the returned value is used in the caller.
 So ideally we should return null in the base impl and ensure the caller is handling it. 

> Add CP hooks for ReplicationEndPoint
> ------------------------------------
>
>                 Key: HBASE-11920
>                 URL: https://issues.apache.org/jira/browse/HBASE-11920
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors, Replication
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 2.0.0, 0.99.1
>
>         Attachments: HBASE-11920.patch, HBASE-11920_1.patch, HBASE-11920_2.patch, HBASE-11920_3.patch,
HBASE-11920_4.patch
>
>
> If we want to create internal replication endpoints other than the one created thro configuration
we may need new hooks. This is something like an internal scanner that we create during compaction
so that the actual compaction scanner can be used as a delegator.
> [~enis]
> If I can give a patch by tomorrow will it be possible to include in the RC?



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

Mime
View raw message