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

Ted Yu commented on HBASE-10985:
--------------------------------

{code}
+public interface SplitTransactionConsensus {
{code}
Please add annotation for audience.
{code}
+   * Finish off split transaction, transition the zknode
{code}
Consensus shouldn't be concerned with zk node, right ?
Same with znodeVersion below:
{code}
+      HRegion first, HRegion second, int znodeVersion, HRegion parent)
{code}
SplitTransactionConsensus shows, in various places, the dependence on zookeeper.

> Decouple Split Transaction from Zookeeper
> -----------------------------------------
>
>                 Key: HBASE-10985
>                 URL: https://issues.apache.org/jira/browse/HBASE-10985
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Consensus, Zookeeper
>            Reporter: Sergey Soldatov
>         Attachments: HBASE-10985.patch, HBASE-10985.patch, HBASE-10985.patch
>
>
> As part of  HBASE-10296 SplitTransaction should be decoupled from Zookeeper. 
> This is an initial patch for review. At the moment the consensus provider  
> placed directly to SplitTransaction to minimize affected code. In the ideal 
> world it should be done in HServer.



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

Reply via email to