ndimiduk commented on a change in pull request #3536:
URL: https://github.com/apache/hbase/pull/3536#discussion_r686151978
##########
File path:
hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminClient.java
##########
@@ -27,6 +27,7 @@
import java.util.Set;
import java.util.stream.Collectors;
+import org.apache.hadoop.hbase.BalanceRequest;
Review comment:
I'm not an expert in the RSGroupAdmin APIs, let me take a closer look...
I see what you mean. All these Java API methods accept a group name string
in one form or another. In this context, your implementation using the base
`BalanceRequest` makes sense. I guess it's better to keep it as you had it
before my comment.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
