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_r383986883
 
 

 ##########
 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:
   Ok. So, your argument is that folks using current data structures and 
methods will have an easier time if the names of methods, etc., have same basic 
pattern in the new context.
   
   Ok.

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