[ https://issues.apache.org/jira/browse/HBASE-21816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758674#comment-16758674 ]
Karthik Palanisamy commented on HBASE-21816: -------------------------------------------- Thank you [~elserj] for reviewing it. Yes, This should be a simple change to reverse it. I lined edge :) > Print source cluster replication config directory > ------------------------------------------------- > > Key: HBASE-21816 > URL: https://issues.apache.org/jira/browse/HBASE-21816 > Project: HBase > Issue Type: Improvement > Components: Replication > Affects Versions: 3.0.0, 2.0.0 > Environment: NA > Reporter: Karthik Palanisamy > Assignee: Karthik Palanisamy > Priority: Major > Fix For: 3.0.0 > > Attachments: HBASE-21816-001.patch, HBASE-21816-002.patch, > HBASE-21816-003.patch > > > User may get confused, to understanding our HBase configurations which are > loaded for replication. Sometimes, User may place source and destination > cluster conf under "/etc/hbase/conf" directory. It will create uncertainty > because our log points that all the configurations are co-located. > > Existing Logs, > {code:java} > INFO [RpcServer.FifoWFPBQ.replication.handler=2,queue=0,port=16020] > regionserver.DefaultSourceFSConfigurationProvider: Loading source cluster > HDP1 file system configurations from xml files under directory > /etc/hbase/conf/ > {code} > But it should be something like, > {code:java} > INFO [RpcServer.FifoWFPBQ.replication.handler=2,queue=0,port=16020] > regionserver.DefaultSourceFSConfigurationProvider: Loading source cluster > HDP1 file system configurations from xml files under directory > /etc/hbase/conf/HDP1 > {code} > > This jira only to change the log-line, no issue with the functionality. > {code:java} > File confDir = new File(replicationConfDir, replicationClusterId); > String[] listofConfFiles = FileUtil.list(confDir); > for (String confFile : listofConfFiles) { > if (new File(confDir, confFile).isFile() && confFile.endsWith(XML)) { > // Add all the user provided client conf files > sourceClusterConf.addResource(new Path(confDir.getPath(), confFile)); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)