Skip to content

Commit 75844c8

Browse files
shahrs87virajjasani
authored andcommitted
HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (#3577)
Signed-off-by: Viraj Jasani <vjasani@apache.org>
1 parent 0ff6a75 commit 75844c8

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2593,7 +2593,7 @@ public List<ServerName> getExcludedServersForSystemTable() {
25932593
serverList.add(new Pair<>(s, server.getRegionServerVersion(s)));
25942594
}
25952595
if (serverList.isEmpty()) {
2596-
return Collections.emptyList();
2596+
return new ArrayList<>();
25972597
}
25982598
String highestVersion = Collections.max(serverList,
25992599
new Comparator<Pair<ServerName, String>>() {
@@ -2606,7 +2606,7 @@ public int compare(Pair<ServerName, String> o1, Pair<ServerName, String> o2) {
26062606
int comparedValue = VersionInfo.compareVersion(minVersionToMoveSysTables,
26072607
highestVersion);
26082608
if (comparedValue > 0) {
2609-
return Collections.emptyList();
2609+
return new ArrayList<>();
26102610
}
26112611
}
26122612
List<ServerName> res = new ArrayList<>();

0 commit comments

Comments
 (0)