[ https://issues.apache.org/jira/browse/HBASE-12867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14319379#comment-14319379 ]
Hudson commented on HBASE-12867: -------------------------------- FAILURE: Integrated in HBase-1.0 #734 (See [https://builds.apache.org/job/HBase-1.0/734/]) HBASE-12867 Add ability to specify custom replication endpoint to add_peer (enis: rev 6e78ab4af3924366cfc707d1642f6a200b347b18) * hbase-shell/src/test/ruby/test_helper.rb * hbase-shell/src/main/ruby/shell/commands/add_peer.rb * hbase-shell/src/main/ruby/hbase/replication_admin.rb * hbase-shell/src/main/ruby/hbase.rb * hbase-shell/src/test/ruby/hbase/replication_admin_test.rb > Shell does not support custom replication endpoint specification > ---------------------------------------------------------------- > > Key: HBASE-12867 > URL: https://issues.apache.org/jira/browse/HBASE-12867 > Project: HBase > Issue Type: Bug > Reporter: Andrew Purtell > Assignee: Kevin Risden > Labels: beginner, beginners > Fix For: 2.0.0, 1.0.1, 1.1.0 > > Attachments: HBASE-12867-v1.patch, HBASE-12867-v2.patch, > HBASE-12867-v3.patch, HBASE-12867-v3.patch, HBASE-12867-v3.patch, > HBASE-12867.patch > > > On HBASE-12254 and also at > https://github.com/risdenk/hbase-custom-replication-endpoint-example > [~risdenk] made the following observations and suggestions regarding custom > replication endpoints that I think are a reasonable blueprint for improvement: > {quote} > I was trying out the pluggable replication endpoint feature and found the > following: > - you must use the ReplicationAdmin to add the new ReplicationEndpoint > - hbase shell add_peer command doesn't support specifying a custom class > - hbase shell add_peer relies on the newly deprecated ReplicationAdmin > addPeer methods > - ReplicationAdmin addPeer tableCfs is now a Map<TableName, ? extends > Collection<String>> instead of a string > {quote} > We should fix the add_peer command in the shell at least. -- This message was sent by Atlassian JIRA (v6.3.4#6332)