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

Mikhail Antonov commented on HBASE-11025:
-----------------------------------------

I haven't really seen dependency injection widely used in HBase codebase, and 
from hbase-default.xml it appears that current practice to hook up all 
pluggable things (like status publisher, load balancer etc) is via loading them 
manually by full class name?

Regarding the naming - I'll rename the field to "consensusProvider", as it's 
actually what it does - will have methods to retrieve the implementation of 
interface for consensus/coordination functionality for the certain area needed 
coordination.

https://reviews.apache.org/r/20337/ - actually, here is review request for 
patch for HBASE-10915 (which this one is part of, just factored out), and there 
are examples of actually doing the coordination for closing regions on the RS 
side. And some in-depth questions/comments too.

> Infrastructure for pluggable consensus service
> ----------------------------------------------
>
>                 Key: HBASE-11025
>                 URL: https://issues.apache.org/jira/browse/HBASE-11025
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Zookeeper
>            Reporter: Mikhail Antonov
>            Assignee: Mikhail Antonov
>         Attachments: HBASE-11025.patch
>
>
> Related to HBASE-10915.
> In this jira I will extract the changed for property changes, factory and 
> consensus provider interface (+ZK implementation), as it's going to be 
> required by other subtasks of HBASE-10909.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to