You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by GitBox <gi...@apache.org> on 2020/05/28 07:31:45 UTC

[GitHub] [hbase] ddupg commented on a change in pull request #1789: HBASE-22738 Fallback to default group to choose RS when there are no …

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



##########
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:
       Yes, we can config multiple comma-separated 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:
users@infra.apache.org