From 0a20dd47d1abbf0036896fac03dc7d801ebcd5bd Mon Sep 17 00:00:00 2001 From: David Smiley Date: Wed, 4 May 2016 16:03:56 -0400 Subject: [PATCH] SOLR-5750: Fix test to specify the collection on commit --- .../org/apache/solr/cloud/TestCloudBackupRestore.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/solr/core/src/test/org/apache/solr/cloud/TestCloudBackupRestore.java b/solr/core/src/test/org/apache/solr/cloud/TestCloudBackupRestore.java index 0af9e648f4f..2dba20b552e 100644 --- a/solr/core/src/test/org/apache/solr/cloud/TestCloudBackupRestore.java +++ b/solr/core/src/test/org/apache/solr/cloud/TestCloudBackupRestore.java @@ -87,7 +87,9 @@ public class TestCloudBackupRestore extends SolrCloudTestCase { } } - create.process(cluster.getSolrClient()); + CloudSolrClient solrClient = cluster.getSolrClient(); + create.process(solrClient); + indexDocs(collectionName); if (!isImplicit && random().nextBoolean()) { @@ -95,14 +97,14 @@ public class TestCloudBackupRestore extends SolrCloudTestCase { int prevActiveSliceCount = getActiveSliceCount(collectionName); CollectionAdminRequest.SplitShard splitShard = CollectionAdminRequest.splitShard(collectionName); splitShard.setShardName("shard1"); - splitShard.process(cluster.getSolrClient()); + splitShard.process(solrClient); // wait until we see one more active slice... for (int i = 0; getActiveSliceCount(collectionName) != prevActiveSliceCount + 1; i++) { assertTrue(i < 30); Thread.sleep(500); } // issue a hard commit. Split shard does a soft commit which isn't good enough for the backup/snapshooter to see - cluster.getSolrClient().commit(); + solrClient.commit(collectionName); } testBackupAndRestore(collectionName); @@ -120,7 +122,6 @@ public class TestCloudBackupRestore extends SolrCloudTestCase { return; } CloudSolrClient client = cluster.getSolrClient(); - client.setDefaultCollection(collectionName); List docs = new ArrayList<>(numDocs); for (int i=0; i