Skip to content

Commit d307a8f

Browse files
authored
HBASE-26185 Return mutable list in AssignmentManager#getExcludedServersForSystemTable (apache#3577)
Signed-off-by: Viraj Jasani <vjasani@apache.org>
1 parent 5e8a269 commit d307a8f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2322,15 +2322,15 @@ public List<ServerName> getExcludedServersForSystemTable() {
23222322
.map(s->new Pair<>(s, master.getRegionServerVersion(s)))
23232323
.collect(Collectors.toList());
23242324
if (serverList.isEmpty()) {
2325-
return Collections.emptyList();
2325+
return new ArrayList<>();
23262326
}
23272327
String highestVersion = Collections.max(serverList,
23282328
(o1, o2) -> VersionInfo.compareVersion(o1.getSecond(), o2.getSecond())).getSecond();
23292329
if (!DEFAULT_MIN_VERSION_MOVE_SYS_TABLES_CONFIG.equals(minVersionToMoveSysTables)) {
23302330
int comparedValue = VersionInfo.compareVersion(minVersionToMoveSysTables,
23312331
highestVersion);
23322332
if (comparedValue > 0) {
2333-
return Collections.emptyList();
2333+
return new ArrayList<>();
23342334
}
23352335
}
23362336
return serverList.stream()

0 commit comments

Comments
 (0)