infraio commented on a change in pull request #1789:
URL: https://github.com/apache/hbase/pull/1789#discussion_r431571294



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java
##########
@@ -77,6 +81,23 @@
   private volatile RSGroupInfoManager rsGroupInfoManager;
   private LoadBalancer internalBalancer;
 
+  /** Define the config key of if fallback group is enabled */
+  public static final String FALLBACK_GROUP_ENABLED_KEY = 
"hbase.rsgroup.fallback.groups.enable";
+  public static final boolean FALLBACK_GROUP_ENABLED_DEFAULT = false;
+
+  /** Define the config key of fallback groups */
+  public static final String FALLBACK_GROUPS_KEY = 
"hbase.rsgroup.fallback.groups";

Review comment:
       groups? Allow fallback to more than one group?




----------------------------------------------------------------
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