You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@solr.apache.org by ds...@apache.org on 2025/09/09 02:29:48 UTC
(solr) 02/02: Fix flaky test TestCollapseQParserPlugin.testMultiSort (#3609)
This is an automated email from the ASF dual-hosted git repository.
dsmiley pushed a commit to branch branch_9x
in repository https://gitbox.apache.org/repos/asf/solr.git
commit 96c1f47d1708bcff39058600a8238f511c82a349
Author: David Smiley <ds...@apache.org>
AuthorDate: Mon Sep 8 21:58:26 2025 -0400
Fix flaky test TestCollapseQParserPlugin.testMultiSort (#3609)
Relying on index order is unreliable.
(cherry picked from commit a08e8cf1d0647339cdf5c6db5c349b2aaaee2b0a)
---
.../org/apache/solr/search/TestCollapseQParserPlugin.java | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java b/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
index 962186fd8b2..716a4a129bc 100644
--- a/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
+++ b/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
@@ -107,17 +107,6 @@ public class TestCollapseQParserPlugin extends SolrTestCaseJ4 {
"//result/doc[1]/str[@name='id'][.='8']",
"//result/doc[2]/str[@name='id'][.='4']");
- // tie broken by index order
- params = new ModifiableSolrParams();
- params.add("q", "*:*");
- params.add("fq", "{!collapse field=group_s sort='test_l desc'}");
- params.add("sort", "id_i desc");
- assertQ(
- req(params),
- "*[count(//doc)=2]",
- "//result/doc[1]/str[@name='id'][.='6']",
- "//result/doc[2]/str[@name='id'][.='2']");
-
// score, then tiebreakers; note top level sort by score ASCENDING (just for weirdness)
params = new ModifiableSolrParams();
params.add("q", "*:* term_s:YYYY");