You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@solr.apache.org by ho...@apache.org on 2023/05/26 01:32:32 UTC

[solr] branch branch_9x updated: SOLR-16819: Fix HttpPartitionTest

This is an automated email from the ASF dual-hosted git repository.

hossman pushed a commit to branch branch_9x
in repository https://gitbox.apache.org/repos/asf/solr.git


The following commit(s) were added to refs/heads/branch_9x by this push:
     new bfc1f17063f SOLR-16819: Fix HttpPartitionTest
bfc1f17063f is described below

commit bfc1f17063f9cdc786e594269acabccc18a480f2
Author: Chris Hostetter <ho...@apache.org>
AuthorDate: Thu May 25 17:30:36 2023 -0700

    SOLR-16819: Fix HttpPartitionTest
    
    Completes partial changes made in SOLR-10466
    
    (cherry picked from commit 358d01d3354e2489d8399f800b2cc1a6d156ae50)
---
 solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java b/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
index b14ec3dd5ae..c1092da34c7 100644
--- a/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
@@ -338,7 +338,7 @@ public class HttpPartitionTest extends AbstractFullDistribZkTestBase {
     String testCollectionName = "c8n_1x3";
     createCollectionRetry(testCollectionName, "conf1", 1, 3);
 
-    sendDoc(1);
+    sendDoc(testCollectionName, 1);
 
     List<Replica> notLeaders =
         ensureAllReplicasAreActive(testCollectionName, "shard1", 1, 3, maxWaitSecsToSeeAllActive);
@@ -382,7 +382,7 @@ public class HttpPartitionTest extends AbstractFullDistribZkTestBase {
     notLeaders =
         ensureAllReplicasAreActive(testCollectionName, "shard1", 1, 3, maxWaitSecsToSeeAllActive);
 
-    sendDoc(4);
+    sendDoc(testCollectionName, 4);
 
     assertDocsExistInAllReplicas(notLeaders, testCollectionName, 1, 4);
 
@@ -555,10 +555,6 @@ public class HttpPartitionTest extends AbstractFullDistribZkTestBase {
     }
   }
 
-  protected int sendDoc(int docId) throws Exception {
-    return sendDoc(docId, cloudClient, cloudClient.getDefaultCollection());
-  }
-
   protected int sendDoc(String collectionName, int docId) throws Exception {
     return sendDoc(docId, cloudClient, collectionName);
   }