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/10/11 07:43:13 UTC
lucene-solr:branch_7x: SOLR-11452:
TestTlogReplica.testOnlyLeaderIndexes() failure
Repository: lucene-solr
Updated Branches:
refs/heads/branch_7x 2f3a6c1fb -> 4f650c96a
SOLR-11452: TestTlogReplica.testOnlyLeaderIndexes() failure
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/4f650c96
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/4f650c96
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/4f650c96
Branch: refs/heads/branch_7x
Commit: 4f650c96a60dafaa1a79515b2335a6e21b49b811
Parents: 2f3a6c1
Author: Cao Manh Dat <da...@apache.org>
Authored: Tue Oct 10 10:11:59 2017 +0700
Committer: Cao Manh Dat <da...@apache.org>
Committed: Wed Oct 11 14:42:58 2017 +0700
----------------------------------------------------------------------
.../src/test/org/apache/solr/cloud/TestTlogReplica.java | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/4f650c96/solr/core/src/test/org/apache/solr/cloud/TestTlogReplica.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/TestTlogReplica.java b/solr/core/src/test/org/apache/solr/cloud/TestTlogReplica.java
index 79dce4d..c978a5f 100644
--- a/solr/core/src/test/org/apache/solr/cloud/TestTlogReplica.java
+++ b/solr/core/src/test/org/apache/solr/cloud/TestTlogReplica.java
@@ -489,16 +489,23 @@ public class TestTlogReplica extends SolrCloudTestCase {
waitForNumDocsInAllActiveReplicas(2);
assertCopyOverOldUpdates(1, timeCopyOverPerCores);
+ boolean firstCommit = true;
// UpdateLog copy over old updates
for (int i = 15; i <= 150; i++) {
cloudClient.add(collectionName, sdoc("id",String.valueOf(i)));
if (random().nextInt(100) < 15 & i != 150) {
+ if (firstCommit) {
+ // because tlog replicas periodically ask leader for new segments,
+ // therefore the copy over old updates action must not be triggered until
+ // tlog replicas actually get new segments
+ assertCopyOverOldUpdates(1, timeCopyOverPerCores);
+ firstCommit = false;
+ }
cloudClient.commit(collectionName);
}
}
checkRTG(120,150, cluster.getJettySolrRunners());
waitForReplicasCatchUp(20);
- assertCopyOverOldUpdates(2, timeCopyOverPerCores);
}
@SuppressWarnings("unchecked")