[ https://issues.apache.org/jira/browse/HBASE-4131?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13114914#comment-13114914 ]
Ted Yu commented on HBASE-4131: ------------------------------- Minor comment for Replication.java: {code} + this.zkHelper = new ReplicationZookeeper(server, this.replicating); + } catch (KeeperException ke) { + throw new IOException("Failed replication handler create", ke); + } {code} I think the exception message should include this.replicating This can be done at time of integration. Nice work Dhruba - looking forward to the follow-on JIRA(s) > Make the Replication Service pluggable via a standard interface definition > -------------------------------------------------------------------------- > > Key: HBASE-4131 > URL: https://issues.apache.org/jira/browse/HBASE-4131 > Project: HBase > Issue Type: Improvement > Components: regionserver > Reporter: dhruba borthakur > Assignee: dhruba borthakur > Fix For: 0.94.0 > > Attachments: 4131-backedout.txt, replicationInterface1.txt, > replicationInterface2.txt, replicationInterface3.txt, > replicationInterface4.txt > > > The current HBase code supports a replication service that can be used to > sync data from from one hbase cluster to another. It would be nice to make it > a pluggable interface so that other cross-data-center replication services > can be used in conjuction with HBase. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira