cpoerschke opened a new issue, #13018:
URL: https://github.com/apache/lucene/issues/13018

   ### Description
   
   * The `TermsIncludingScoreQuery` constructor as per 
https://github.com/apache/lucene/blob/releases/lucene/9.9.1/lucene/join/src/java/org/apache/lucene/search/join/TermsIncludingScoreQuery.java#L74
 does a `BytesRefHash.sort()` call on the `terms` object passed in to it.
   * Subsequently `TermsIncludingScoreQuery.createWeight` as per 
https://github.com/apache/lucene/blob/releases/lucene/9.9.1/lucene/join/src/java/org/apache/lucene/search/join/TermsIncludingScoreQuery.java#L134
 passes the `terms` object to the `TermsQuery` constructor.
   * The `TermsQuery` constructor as per 
https://github.com/apache/lucene/blob/releases/lucene/9.9.1/lucene/join/src/java/org/apache/lucene/search/join/TermsQuery.java#L68
 also does a `BytesRefHash.sort()` call on the `terms` object passed in to it.
   
   The current Lucene tests pass but on the dev mailing as per 
https://lists.apache.org/thread/ms65gkmm86cpp5ntdo555ob8mrgg64o5 test failures 
for `org.apache.solr.search.join.ScoreJoinQParserPlugin` were reported and it 
was mentioned that after the #12784 changes `BytesRefHash#sort` may no longer 
be called more than once.
   
   
   ### Version and environment details
   
   _No response_


-- 
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.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