shubhamvishu opened a new pull request, #13134:
URL: https://github.com/apache/lucene/pull/13134

   ### Description
   
   Fixes the `TestKnnByteVectorQuery.testToString` by removing the assumption 
in the test that the first doc has the max score whereas in 
[`AbstractKnnVectorQuery#createRewrittenQuery`](https://github.com/apache/lucene/blob/ca06693a16321eef83617b8e4aabbc7d2f6c5187/lucene/core/src/test/org/apache/lucene/search/TestKnnByteVectorQuery.java#L72)
 we are sorting the docs based on doc id and not score.
   
   Test : Passes the failing seed
   
   ```
   ./gradlew :lucene:core:test --tests 
"org.apache.lucene.search.TestKnnByteVectorQuery.testToString" 
-Ptests.seed=628DBF68334757BB -Ptests.nightly=true
   ```
   
   Closes #13131
   
   <!--
   If this is your first contribution to Lucene, please make sure you have 
reviewed the contribution guide.
   https://github.com/apache/lucene/blob/main/CONTRIBUTING.md
   -->
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to