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 <[email protected]> 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");
