infraio commented on a change in pull request #813: HBASE-23253 Rewrite rsgroup
related UTs with the new methods introduc…
URL: https://github.com/apache/hbase/pull/813#discussion_r346081886
##########
File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/Admin.java
##########
@@ -2315,15 +2315,15 @@ boolean snapshotCleanupSwitch(final boolean on, final
boolean synchronous)
* @param servers set of servers to remove
* @throws IOException if a remote or network exception occurs
*/
- void removeRSGroup(Set<Address> servers) throws IOException;
+ void removeServers(Set<Address> servers) 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 moveToRSGroup(Set<Address> servers, String targetGroup) throws
IOException;
+ void moveServers(Set<Address> servers, String targetGroup) throws
IOException;
Review comment:
moveServersToRSGroup?
----------------------------------------------------------------
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
