HBASE-22764 Fixed remaining Checkstyle issues in the hbase-rsgroup module
Signed-off-by: stack <stack@apache.org>
This commit is contained in:
parent
46a432386e
commit
124fa3488a
|
@ -112,8 +112,7 @@ public class RSGroupAdminServer implements RSGroupAdmin {
|
||||||
* Check passed name. Fail if nulls or if corresponding RSGroupInfo not found.
|
* Check passed name. Fail if nulls or if corresponding RSGroupInfo not found.
|
||||||
* @return The RSGroupInfo named <code>name</code>
|
* @return The RSGroupInfo named <code>name</code>
|
||||||
*/
|
*/
|
||||||
private RSGroupInfo getAndCheckRSGroupInfo(String name)
|
private RSGroupInfo getAndCheckRSGroupInfo(String name) throws IOException {
|
||||||
throws IOException {
|
|
||||||
if (StringUtils.isEmpty(name)) {
|
if (StringUtils.isEmpty(name)) {
|
||||||
throw new ConstraintException("RSGroup cannot be null.");
|
throw new ConstraintException("RSGroup cannot be null.");
|
||||||
}
|
}
|
||||||
|
@ -329,8 +328,7 @@ public class RSGroupAdminServer implements RSGroupAdmin {
|
||||||
value="RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE",
|
value="RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE",
|
||||||
justification="Ignoring complaint because don't know what it is complaining about")
|
justification="Ignoring complaint because don't know what it is complaining about")
|
||||||
@Override
|
@Override
|
||||||
public void moveServers(Set<Address> servers, String targetGroupName)
|
public void moveServers(Set<Address> servers, String targetGroupName) throws IOException {
|
||||||
throws IOException {
|
|
||||||
if (servers == null) {
|
if (servers == null) {
|
||||||
throw new ConstraintException("The list of servers to move cannot be null.");
|
throw new ConstraintException("The list of servers to move cannot be null.");
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue