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

Andrew Purtell edited comment on HBASE-19131 at 11/5/17 6:23 PM:
-----------------------------------------------------------------

I have reverted this from branch-1.4. It should probably be reverted from 
branch-1 too but we can discuss it. The rest of the changes look ok. Put back 
support for this RPC, but deprecate it, and my concern will be resolved. Since 
we would be keeping the old RPC for compatibility we should also keep around 
the related coprocessor hooks. Mark them as deprecated in branch-1. No need to 
keep them around in branch-2. Fine to add the new coprocessor hooks 
preGetClusterStatus / postGetClusterStatus




was (Author: apurtell):
I have reverted this from branch-1.4. It should probably be reverted from 
branch-1 too but we can discuss it. The rest of the changes look ok. Put back 
support for this RPC, but deprecate it, and my concern will be resolved.

> Add the ClusterStatus hook and cleanup other hooks which can be replaced by 
> ClusterStatus hook
> ----------------------------------------------------------------------------------------------
>
>                 Key: HBASE-19131
>                 URL: https://issues.apache.org/jira/browse/HBASE-19131
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Chia-Ping Tsai
>            Assignee: Chia-Ping Tsai
>             Fix For: 1.4.0, 1.5.0, 2.0.0-beta-1
>
>         Attachments: HBASE-19131.branch-1.v0.patch, 
> HBASE-19131.branch-1.v1.patch, HBASE-19131.v0.patch, HBASE-19131.v1.patch, 
> HBASE-19131.v2.patch, HBASE-19131.v3.patch, HBASE-19131.v4.patch
>
>
> This issue will add the hook for {{ClusterStatus}} and cleanup other hooks 
> which can be replaced by {{ClusterStatus}} hook.
> # {{ListDeadServers}} -  introduced by HBASE-18131



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to