saintstack commented on a change in pull request #3242:
URL: https://github.com/apache/hbase/pull/3242#discussion_r629480082



##########
File path: 
hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestingUtility.java
##########
@@ -58,8 +58,9 @@ public IntegrationTestingUtility(Configuration conf) {
    */
   public static final String IS_DISTRIBUTED_CLUSTER = 
"hbase.test.cluster.distributed";
 
-  /** Config for pluggable hbase cluster manager */
-  private static final String HBASE_CLUSTER_MANAGER_CLASS = 
"hbase.it.clustermanager.class";
+  /** Config for pluggable hbase cluster manager. Pass fully-qualified class 
name as property
+   * value. Drop the '.class' suffix.*/
+  public static final String HBASE_CLUSTER_MANAGER_CLASS = 
"hbase.it.clustermanager.class";

Review comment:
       I have an implementation in an external repo; I want to refer to this 
static define.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to