[ 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)