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/16 17:03:34 UTC

[2/2] lucene-solr:master: 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/d2f7272b
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/d2f7272b
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/d2f7272b

Branch: refs/heads/master
Commit: d2f7272b0e2d4faefdf0c7c7e991311bde4e9c3e
Parents: 15002eb
Author: Christine Poerschke <cp...@apache.org>
Authored: Tue Oct 16 12:54:00 2018 -0400
Committer: Christine Poerschke <cp...@apache.org>
Committed: Tue Oct 16 12:54:00 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/d2f7272b/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;
   }