[ https://issues.apache.org/jira/browse/HBASE-7441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13546280#comment-13546280 ]
stack commented on HBASE-7441: ------------------------------ [~liushaohui] Patch looks good but I would not add the define HBASE_CLUSTER_MANAGER_CLASS in hbase-common. Keep the define in the class where it is actually used; i.e. in hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestingUtility.java > Make ClusterManager in IntegrationTestingUtility pluggable > ---------------------------------------------------------- > > Key: HBASE-7441 > URL: https://issues.apache.org/jira/browse/HBASE-7441 > Project: HBase > Issue Type: Improvement > Components: test > Affects Versions: 0.94.3 > Reporter: Liu Shaohui > Assignee: Liu Shaohui > Priority: Minor > Labels: newbie, patch > Fix For: 0.96.0, 0.94.5 > > Attachments: HBASE-7441-0.94-v1.patch, HBASE-7441-trunk-v1.patch > > Original Estimate: 3h > Remaining Estimate: 3h > > After the patch HBASE-7009, we can use ChaosMonkey to test the Hbase cluster. > The ClusterManager use ssh to stop/start the rs or master without passwd. To > support other cluster manager tool, we need to make clusterManager in > IntegrationTestingUtility pluggable. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira