From 8c197f20739cf4a5d002686fd0304fc7a525b721 Mon Sep 17 00:00:00 2001 From: Sammi Chen Date: Wed, 31 Jul 2019 20:19:38 +0800 Subject: [PATCH] HDDS-1882. TestReplicationManager failed with NPE --- .../hadoop/hdds/scm/container/ReplicationManager.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java index 0557ea10d209b..a8dff405e4386 100644 --- a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java +++ b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ReplicationManager.java @@ -486,8 +486,11 @@ private void handleUnderReplicatedContainer(final ContainerInfo container, final List excludeList = replicas.stream() .map(ContainerReplica::getDatanodeDetails) .collect(Collectors.toList()); - inflightReplication.get(id).stream().map(r -> r.datanode) - .forEach(excludeList::add); + List actionList = inflightReplication.get(id); + if (actionList != null) { + actionList.stream().map(r -> r.datanode) + .forEach(excludeList::add); + } final List selectedDatanodes = containerPlacement .chooseDatanodes(excludeList, null, delta, container.getUsedBytes());