[hbase] branch branch-2.3 updated: HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (#3577)

[vjasani]

This is an automated email from the ASF dual-hosted git repository.

vjasani pushed a commit to branch branch-2.3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.3 by this push:
     new 3ee40e0  HBASE-26185 Return mutable list in 
AssignmentManager#getExcludedServersForSystemTable (#3577)
3ee40e0 is described below

commit 3ee40e001fecfa00e878d3b8a79774b624cc3cbd
Author: Rushabh Shah <shahr...@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 <vjas...@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 b51365a..d2393fe 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
@@ -2281,7 +2281,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();
@@ -2289,7 +2289,7 @@ public class AssignmentManager {
       int comparedValue = VersionInfo.compareVersion(minVersionToMoveSysTables,
         highestVersion);
       if (comparedValue > 0) {
-        return Collections.emptyList();
+        return new ArrayList<>();
       }
     }
     return serverList.stream()