You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by da...@apache.org on 2018/05/23 02:30:56 UTC
lucene-solr:branch_7x: SOLR-12247:
NodeAddedTriggerTest.testRestoreState() failure: Did not expect the processor
to fire on first run
Repository: lucene-solr
Updated Branches:
refs/heads/branch_7x 982268efd -> 8db3912ca
SOLR-12247: NodeAddedTriggerTest.testRestoreState() failure: Did not expect the processor to fire on first run
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/8db3912c
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/8db3912c
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/8db3912c
Branch: refs/heads/branch_7x
Commit: 8db3912cac9ba73ff391cb1e9eead0c527f018f8
Parents: 982268e
Author: Cao Manh Dat <da...@apache.org>
Authored: Wed May 23 09:30:19 2018 +0700
Committer: Cao Manh Dat <da...@apache.org>
Committed: Wed May 23 09:30:46 2018 +0700
----------------------------------------------------------------------
.../org/apache/solr/cloud/autoscaling/NodeAddedTriggerTest.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/8db3912c/solr/core/src/test/org/apache/solr/cloud/autoscaling/NodeAddedTriggerTest.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/autoscaling/NodeAddedTriggerTest.java b/solr/core/src/test/org/apache/solr/cloud/autoscaling/NodeAddedTriggerTest.java
index cd52785..183d154 100644
--- a/solr/core/src/test/org/apache/solr/cloud/autoscaling/NodeAddedTriggerTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/autoscaling/NodeAddedTriggerTest.java
@@ -254,6 +254,7 @@ public class NodeAddedTriggerTest extends SolrCloudTestCase {
trigger.run();
JettySolrRunner newNode = cluster.startJettySolrRunner();
+ trigger.setProcessor(null); // the processor may get called for old nodes
trigger.run(); // this run should detect the new node
trigger.close(); // close the old trigger
@@ -301,7 +302,9 @@ public class NodeAddedTriggerTest extends SolrCloudTestCase {
assertTrue(fired.get());
TriggerEvent nodeAddedEvent = eventRef.get();
assertNotNull(nodeAddedEvent);
- //TODO assertEquals("", newNode.getNodeName(), nodeAddedEvent.getProperty(NodeAddedTrigger.NodeAddedEvent.NODE_NAME));
+ List<String> nodeNames = (List<String>) nodeAddedEvent.getProperty(NodeAddedTrigger.NodeAddedEvent.NODE_NAMES);
+ assertTrue("Newly added node was not present in event message",
+ nodeNames.contains(newNode.getNodeName()));
}
}