[hbase] branch branch-1 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-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 <shahr...@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 <vjas...@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<>();