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

Anoop Sam John commented on HBASE-16769:
----------------------------------------

Master CP hooks are around snapshot and quota features. The PB objects here are 
no large sized.  We do have our POJO - PB and vice versa conversion.
But RSO seems not so easy.  We pass the replicating WALEntry PB objects and 
cells to hook.  Cells are passed as CellScanner so no PB stuff here.  But there 
will be many WALEntry objects. For each we need to make either our POJO or 
unshaded PB objects seems lot of work and garbage.  We should not be passing 
the WALEntries and Cells to this hooks? These hooks are added by HBASE-12916 
(For AC check on replication APIs).  There any way these params not so imp.  
wdyt?

> Deprecate/remove PB references from MasterObserver and RegionServerObserver
> ---------------------------------------------------------------------------
>
>                 Key: HBASE-16769
>                 URL: https://issues.apache.org/jira/browse/HBASE-16769
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>            Priority: Blocker
>             Fix For: 2.0.0
>
>
> This is effectively a sub-task for HBASE-15174.
> CP Methods
> MasterObserver
>       preListSnapshot
>       postListSnapshot
>       preSnapshot
>       postSnapshot
>       preCloneSnapshot
>       postCloneSnapshot
>       preRestoreSnapshot
>       postRestoreSnapshot
>       preDeleteSnapshot
>       postDeleteSnapshot
>       
>       preSetUserQuota
>       postSetUserQuota
>       preSetUserQuota
>       postSetUserQuota
>       preSetUserQuota
>       postSetUserQuota
>       preSetTableQuota
>       postSetTableQuota
>       preSetNamespaceQuota
>       postSetNamespaceQuota
>       
> RegionServerObserver
>       preReplicateLogEntries
>       postReplicateLogEntries



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

Reply via email to