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 2017/12/27 15:04:04 UTC

[11/54] [abbrv] lucene-solr:jira/solr-11702: SOLR-11285: Remove stray logging. Relax test assertion due to timing under load.

SOLR-11285: Remove stray logging. Relax test assertion due to timing under load.


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/d02d1f1c
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/d02d1f1c
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/d02d1f1c

Branch: refs/heads/jira/solr-11702
Commit: d02d1f1cab3fa9191a3b677a64a9ad56d2990089
Parents: 8eb5c69
Author: Andrzej Bialecki <ab...@apache.org>
Authored: Fri Dec 15 11:30:50 2017 +0100
Committer: Andrzej Bialecki <ab...@apache.org>
Committed: Fri Dec 15 11:30:50 2017 +0100

----------------------------------------------------------------------
 .../solr/cloud/autoscaling/sim/SimDistribStateManager.java   | 1 -
 .../solr/cloud/autoscaling/sim/TestTriggerIntegration.java   | 8 ++++----
 2 files changed, 4 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/d02d1f1c/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimDistribStateManager.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimDistribStateManager.java b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimDistribStateManager.java
index f9f17a0..cfa6265 100644
--- a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimDistribStateManager.java
+++ b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimDistribStateManager.java
@@ -303,7 +303,6 @@ public class SimDistribStateManager implements DistribStateManager {
     for (int i = 0; i < elements.length; i++) {
       String currentName = elements[i];
       currentPath.append('/');
-      LOG.info(" - parentNode=" + parentNode + ", currentName=" + currentName);
       n = parentNode.children != null ? parentNode.children.get(currentName) : null;
       if (n == null) {
         if (create) {

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/d02d1f1c/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestTriggerIntegration.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestTriggerIntegration.java b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestTriggerIntegration.java
index a05eb78..3a118f2 100644
--- a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestTriggerIntegration.java
+++ b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/TestTriggerIntegration.java
@@ -1209,9 +1209,9 @@ public class TestTriggerIntegration extends SimSolrCloudTestCase {
     assertEquals(collectionRates.toString(), 1, collectionRates.size());
     Double collectionRate = collectionRates.get(COLL1);
     assertNotNull(collectionRate);
-    assertTrue(collectionRate > 5.0);
-    assertEquals(collectionRate, totalNodeRate.get(), 5.0);
-    assertEquals(collectionRate, totalShardRate.get(), 5.0);
-    assertEquals(collectionRate, totalReplicaRate.get(), 5.0);
+    assertTrue(collectionRate > 100.0);
+    assertTrue(totalNodeRate.get() > 100.0);
+    assertTrue(totalShardRate.get() > 100.0);
+    assertTrue(totalReplicaRate.get() > 100.0);
   }
 }