busbey 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_r383932417
########## 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 +enable rs group feature. + +The main difference comparing to the old implementation is that, now the +rsgroup for a table is stored in TableDescriptor, instead of in RSGroupInfo, +so the getTables method of RSGroupInfo has been deprecated. And if you use the +Admin methods to get the RSGroupInfo, its getTables method will always return +empty. Of course the behavior for the old RSGroupAdminEndpoint is not changed, +we will fill the tables field of the RSGroupInfo before returning, to make it +compatible with old hbase client/shell. + +When upgrading, the migration between the RSGroupInfo and TableDescriptor will Review comment: excellent. please add a sentence here that expressly calls out that enforcement will keep working during the migration and running new admin commands is fine. probably I will get folks asking me how to tell when the migration is complete, so if there's an easy answer to that we should include it. ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
