You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by cp...@apache.org on 2018/10/18 22:17:11 UTC
[4/4] lucene-solr:branch_7x: SOLR-12877: avoid NPE in
TestUtilizeNode.getReplicaList
SOLR-12877: avoid NPE in TestUtilizeNode.getReplicaList
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/4d811b79
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/4d811b79
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/4d811b79
Branch: refs/heads/branch_7x
Commit: 4d811b798976e3213737ffdccd4540e40218d837
Parents: ac60ddd
Author: Christine Poerschke <cp...@apache.org>
Authored: Tue Oct 16 12:54:00 2018 -0400
Committer: Christine Poerschke <cp...@apache.org>
Committed: Thu Oct 18 17:59:44 2018 -0400
----------------------------------------------------------------------
.../src/test/org/apache/solr/cloud/TestUtilizeNode.java | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/4d811b79/solr/core/src/test/org/apache/solr/cloud/TestUtilizeNode.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/TestUtilizeNode.java b/solr/core/src/test/org/apache/solr/cloud/TestUtilizeNode.java
index bc64b6d..18ac662 100644
--- a/solr/core/src/test/org/apache/solr/cloud/TestUtilizeNode.java
+++ b/solr/core/src/test/org/apache/solr/cloud/TestUtilizeNode.java
@@ -170,11 +170,13 @@ public class TestUtilizeNode extends SolrCloudTestCase {
.getClusterState().getCollectionOrNull(collectionName, false);
List<Replica> results = new ArrayList<>(3);
- collection.forEachReplica((s, replica) -> {
+ if (collection != null) {
+ collection.forEachReplica((s, replica) -> {
if (replica.getNodeName().equals(jettyNode.getNodeName())) {
- results.add(replica);
- }
- });
+ results.add(replica);
+ }
+ });
+ }
return results;
}