saintstack commented on a change in pull request #657: Hbase 22932 Add rs group
management methods in Admin and AsyncAdmin
URL: https://github.com/apache/hbase/pull/657#discussion_r328710842
##########
File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/Admin.java
##########
@@ -2254,4 +2256,70 @@ boolean snapshotCleanupSwitch(final boolean on, final
boolean synchronous)
*/
boolean isSnapshotCleanupEnabled() throws IOException;
+ /**
+ * Gets group info for the given group name
+ * @param groupName the group name
+ * @return group info
+ * @throws IOException if a remote or network exception occurs
+ */
+ RSGroupInfo getRSGroupInfo(String groupName) throws IOException;
+
+ /**
+ * Move given set of servers to the specified target RegionServer group
+ * @param servers set of servers to move
+ * @param targetGroup the group to move servers to
+ * @throws IOException if a remote or network exception occurs
+ */
+ void moveServers(Set<Address> servers, String targetGroup) throws
IOException;
Review comment:
Should all RS Grouping commands have a common prefix? You have RSGroup on
some of them but not all such as this one.
----------------------------------------------------------------
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
