Apache9 commented on a change in pull request #1206: HBASE-23890 Update the 
rsgroup section in our ref guide
URL: https://github.com/apache/hbase/pull/1206#discussion_r384428798
 
 

 ##########
 File path: src/main/asciidoc/_chapters/ops_mgt.adoc
 ##########
 @@ -3479,6 +3478,27 @@ To enable ACL, add the following to your hbase-site.xml 
and restart your Master:
 <property>
 ----
 
+=== Migrating From Old Implementation
+The coprocessor `org.apache.hadoop.hbase.rsgroup.RSGroupAdminEndpoint` is
+deprected, but for compatible, if you want the pre 3.0.0 hbase client/shell
+to communicate with the new hbase cluster, you still need to add this
+coprocessor to master. And if this coprocessor is specified, the
+`hbase.balancer.rsgroup.enabled` flag will be set to true automatically to
 
 Review comment:
   OK, we have this in RSGroupBasedLoadBalancer.initialize
   
   ```
       // Create the balancer
       Class<? extends LoadBalancer> balancerClass;
       String balancerClassName = config.get(HBASE_RSGROUP_LOADBALANCER_CLASS);
       if (balancerClassName == null) {
         balancerClass = 
config.getClass(HConstants.HBASE_MASTER_LOADBALANCER_CLASS,
           LoadBalancerFactory.getDefaultLoadBalancerClass(), 
LoadBalancer.class);
       } else {
         try {
           balancerClass = 
Class.forName(balancerClassName).asSubclass(LoadBalancer.class);
         } catch (ClassNotFoundException e) {
           throw new IOException(e);
         }
       }
       // avoid infinite nesting
       if (getClass().isAssignableFrom(balancerClass)) {
         balancerClass = LoadBalancerFactory.getDefaultLoadBalancerClass();
       }
       internalBalancer = ReflectionUtils.newInstance(balancerClass);
   ```

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


With regards,
Apache Git Services

Reply via email to