This is an automated email from the ASF dual-hosted git repository.
psalagnac 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 3745145d3d1 Fix rarely flaky DeleteInactiveReplicaTest. (#3012)
3745145d3d1 is described below
commit 3745145d3d1cea43430daf9a4bd73ee24376e39b
Author: Pierre Salagnac <[email protected]>
AuthorDate: Thu Jan 9 22:47:30 2025 +0100
Fix rarely flaky DeleteInactiveReplicaTest. (#3012)
The test was retrieving a replica from cluster state using core name
instead of correct replica name.
---
.../src/test/org/apache/solr/cloud/DeleteInactiveReplicaTest.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/solr/core/src/test/org/apache/solr/cloud/DeleteInactiveReplicaTest.java
b/solr/core/src/test/org/apache/solr/cloud/DeleteInactiveReplicaTest.java
index 8d3749b7eba..33bab69aeae 100644
--- a/solr/core/src/test/org/apache/solr/cloud/DeleteInactiveReplicaTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/DeleteInactiveReplicaTest.java
@@ -83,7 +83,7 @@ public class DeleteInactiveReplicaTest extends
SolrCloudTestCase {
"Expected replica " + replica.getName() + " on down node to be removed
from cluster state",
collectionName,
(n, c) -> {
- Replica r = c.getReplica(replica.getCoreName());
+ Replica r = c.getReplica(replica.getName());
return r == null || r.getState() != Replica.State.ACTIVE;
});
@@ -96,7 +96,7 @@ public class DeleteInactiveReplicaTest extends
SolrCloudTestCase {
"Expected deleted replica " + replica.getName() + " to be removed from
cluster state",
collectionName,
(n, c) -> {
- return c.getReplica(replica.getCoreName()) == null;
+ return c.getReplica(replica.getName()) == null;
});
cluster.startJettySolrRunner(jetty);