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:21:10 UTC
[hbase] branch branch-2.4 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-2.4
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.4 by this push:
new 9d34fa7 HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (#3577)
9d34fa7 is described below
commit 9d34fa7d5824c2890e5c79e8718937e7a7826f29
Author: Rushabh Shah <sh...@gmail.com>
AuthorDate: Thu Aug 12 04:16:22 2021 -0400
HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (#3577)
Signed-off-by: Viraj Jasani <vj...@apache.org>
---
.../org/apache/hadoop/hbase/master/assignment/AssignmentManager.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
index 2377be6..0bb05e3 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
@@ -2309,7 +2309,7 @@ public class AssignmentManager {
.map(s->new Pair<>(s, master.getRegionServerVersion(s)))
.collect(Collectors.toList());
if (serverList.isEmpty()) {
- return Collections.emptyList();
+ return new ArrayList<>();
}
String highestVersion = Collections.max(serverList,
(o1, o2) -> VersionInfo.compareVersion(o1.getSecond(), o2.getSecond())).getSecond();
@@ -2317,7 +2317,7 @@ public class AssignmentManager {
int comparedValue = VersionInfo.compareVersion(minVersionToMoveSysTables,
highestVersion);
if (comparedValue > 0) {
- return Collections.emptyList();
+ return new ArrayList<>();
}
}
return serverList.stream()