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

Mikhail Antonov commented on HBASE-11092:
-----------------------------------------

Yes.. from the patch:

{noformat}
Index: 
hbase-server/src/main/java/org/apache/hadoop/hbase/consensus/ConsensusProvider.java
===================================================================
--- 
hbase-server/src/main/java/org/apache/hadoop/hbase/consensus/ConsensusProvider.java
 (revision cbd278b45328ed39e143885d4dc2a48f3409af05)
+++ hbase-client/src/main/java/org/apache/hadoop/hbase/ConsensusProvider.java   
(revision ea044ecab8b1a2d87e48e1b8f5456e5831e5a280)
@@ -15,15 +15,15 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.apache.hadoop.hbase.consensus;
+package org.apache.hadoop.hbase;
{noformat}

The patch is gathered from 3 local commits, but the actual consensus provider 
is now in o.a.h.h.

> Server interface should have method getConsensusProvider()
> ----------------------------------------------------------
>
>                 Key: HBASE-11092
>                 URL: https://issues.apache.org/jira/browse/HBASE-11092
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Consensus
>    Affects Versions: 0.99.0
>            Reporter: Mikhail Antonov
>            Assignee: Mikhail Antonov
>             Fix For: 0.99.0
>
>         Attachments: HBASE-11092.patch, HBASE-11092.patch
>
>
> As discussed in comments to HBASE-10915, we need to have a proper way to 
> retrieve instance of consensus provider, and Server interface seems the right 
> one.
> Since Server interface lives in hbase-client maven module, the following 
> approach is implemented in this patch:
>  - hbase-client module has very basic (almost marker) interface 
> ConsensusProvider to return instance of consensus provider from the Server
>  - hbase-server module has BaseConsensusProvider which defines the consensus 
> interfaces
>  - Implementations shall subclass BaseConsensusProvider
>  - whoever wants to get ConsensusProvider from raw Server interface on 
> hbase-server side, has to typecast: (BaseConsensusProvider) 



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

Reply via email to