You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ab...@apache.org on 2018/04/04 15:14:19 UTC
lucene-solr:branch_7x: SOLR-12095: Missed a few calls to init().
Repository: lucene-solr
Updated Branches:
refs/heads/branch_7x 02fa5a0b0 -> 12309659f
SOLR-12095: Missed a few calls to init().
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/12309659
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/12309659
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/12309659
Branch: refs/heads/branch_7x
Commit: 12309659f603f2bbbe353a459eb916adde0c8a02
Parents: 02fa5a0
Author: Andrzej Bialecki <ab...@apache.org>
Authored: Wed Apr 4 17:13:24 2018 +0200
Committer: Andrzej Bialecki <ab...@apache.org>
Committed: Wed Apr 4 17:14:13 2018 +0200
----------------------------------------------------------------------
.../apache/solr/cloud/autoscaling/sim/TestNodeAddedTrigger.java | 4 ++++
1 file changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/12309659/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestNodeAddedTrigger.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestNodeAddedTrigger.java b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestNodeAddedTrigger.java
index fd816ca..04e8c1d 100644
--- a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestNodeAddedTrigger.java
+++ b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestNodeAddedTrigger.java
@@ -80,6 +80,7 @@ public class TestNodeAddedTrigger extends SimSolrCloudTestCase {
try (NodeAddedTrigger trigger = new NodeAddedTrigger("node_added_trigger")) {
trigger.configure(cluster.getLoader(), cluster, props);
+ trigger.init();
trigger.setProcessor(noFirstRunProcessor);
trigger.run();
@@ -250,6 +251,7 @@ public class TestNodeAddedTrigger extends SimSolrCloudTestCase {
// and assert that the new trigger still fires
NodeAddedTrigger trigger = new NodeAddedTrigger("node_added_trigger");
trigger.configure(cluster.getLoader(), cluster, props);
+ trigger.init();
trigger.setProcessor(noFirstRunProcessor);
trigger.run();
@@ -259,6 +261,7 @@ public class TestNodeAddedTrigger extends SimSolrCloudTestCase {
try (NodeAddedTrigger newTrigger = new NodeAddedTrigger("some_different_name")) {
newTrigger.configure(cluster.getLoader(), cluster, props);
+ trigger.init();
try {
newTrigger.restoreState(trigger);
fail("Trigger should only be able to restore state from an old trigger of the same name");
@@ -269,6 +272,7 @@ public class TestNodeAddedTrigger extends SimSolrCloudTestCase {
try (NodeAddedTrigger newTrigger = new NodeAddedTrigger("node_added_trigger")) {
newTrigger.configure(cluster.getLoader(), cluster, props);
+ newTrigger.init();
AtomicBoolean fired = new AtomicBoolean(false);
AtomicReference<TriggerEvent> eventRef = new AtomicReference<>();
newTrigger.setProcessor(event -> {