You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2021/08/12 08:26:00 UTC
[hbase] branch branch-1 updated: HBASE-26185 Return mutable list in
AssignmentManager#getExcludedServersForSystemTable (#3577)
This is an automated email from the ASF dual-hosted git repository.
vjasani pushed a commit to branch branch-1
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-1 by this push:
new 75844c8 HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (#3577)
75844c8 is described below
commit 75844c8c3933b2e10a3eb91a0d1126bed58c1832
Author: Rushabh Shah <sh...@gmail.com>
AuthorDate: Thu Aug 12 13:53:08 2021 +0530
HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (#3577)
Signed-off-by: Viraj Jasani <vj...@apache.org>
---
.../main/java/org/apache/hadoop/hbase/master/AssignmentManager.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
index 83895e8..376ff82 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
@@ -2593,7 +2593,7 @@ public class AssignmentManager extends ZooKeeperListener {
serverList.add(new Pair<>(s, server.getRegionServerVersion(s)));
}
if (serverList.isEmpty()) {
- return Collections.emptyList();
+ return new ArrayList<>();
}
String highestVersion = Collections.max(serverList,
new Comparator<Pair<ServerName, String>>() {
@@ -2606,7 +2606,7 @@ public class AssignmentManager extends ZooKeeperListener {
int comparedValue = VersionInfo.compareVersion(minVersionToMoveSysTables,
highestVersion);
if (comparedValue > 0) {
- return Collections.emptyList();
+ return new ArrayList<>();
}
}
List<ServerName> res = new ArrayList<>();