saintstack commented on a change in pull request #1165: HBASE-22514 Move 
rsgroup feature into core of HBase
URL: https://github.com/apache/hbase/pull/1165#discussion_r378438971
 
 

 ##########
 File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/Admin.java
 ##########
 @@ -2262,4 +2265,104 @@ boolean snapshotCleanupSwitch(final boolean on, final 
boolean synchronous)
    */
   boolean isSnapshotCleanupEnabled() throws IOException;
 
+  /**
+   * Creates a new RegionServer group with the given name
+   * @param groupName the name of the group
+   * @throws IOException if a remote or network exception occurs
+   */
+  void addRSGroup(String groupName) throws IOException;
 
 Review comment:
   Do we still need 'RS'? Should this be addGroup or addRegionServerGroup? 

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