[
https://issues.apache.org/jira/browse/HBASE-23683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17022998#comment-17022998
]
Wellington Chevreuil commented on HBASE-23683:
----------------------------------------------
Had reviewed the PR for master, then realised the addendum I had in mind
doesn't apply, as in master branch, *HBaseInterClusterReplicationEndpoint*
relies on *AsyncClusterConnection*, which has no relation with *Connection*
interface, as I previously thought. So we don't have how to shield potential
extensions of *HBaseInterClusterReplicationEndpoint* from requiring extra
changes once upgrading from 2.x to 3.x, unless we do further refactoring of
*AsyncClusterConnection* to somehow extend *Connection,* so I'm leaving this as
it is now.
> Make HBaseInterClusterReplicationEndpoint more extensible
> ---------------------------------------------------------
>
> Key: HBASE-23683
> URL: https://issues.apache.org/jira/browse/HBASE-23683
> Project: HBase
> Issue Type: Improvement
> Components: Replication
> Reporter: Wellington Chevreuil
> Assignee: Wellington Chevreuil
> Priority: Major
> Fix For: 3.0.0, 2.3.0
>
>
> *HBaseInterClusterReplicationEndpoint* currently creates the cluster
> connection and sink manager instances inside its _init_ method and assigns
> those to private class variables. Then any potential custom extension of
> *HBaseInterClusterReplicationEndpoint* that requires custom implementations
> of connection and/or sink manager would need to resort to _java reflection_
> for effectively replace those instances, such as below:
> {noformat}
> ...
> ClusterConnection conn = (ClusterConnection)ConnectionFactory.
> createConnection(context.getConfiguration(),
> User.create(replicationUgi));
> ReplicationSinkManager sinkManager = new ReplicationSinkManager(conn,
> ctx.getPeerId(),
> this, context.getConfiguration());
> try {
> Field field = this.getClass().getSuperclass().getDeclaredField("conn");
> field.setAccessible(true);
> field.set(this, conn);
> field =
> this.getClass().getSuperclass().getDeclaredField("replicationSinkMgr");
> field.setAccessible(true);
> field.set(this, sinkManager);
> } catch (Exception e) {
> throw new IOException(e);
> }
> ...
> {noformat}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
