Github user jimczi commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/496#discussion_r232800760
  
    --- Diff: 
lucene/core/src/java/org/apache/lucene/search/TopFieldCollector.java ---
    @@ -68,6 +68,18 @@ public void setScorer(Scorable scorer) throws 
IOException {
       }
     
       static boolean canEarlyTerminate(Sort searchSort, Sort indexSort) {
    +    return canEarlyTerminateOnDocId(searchSort, indexSort) ||
    +           canEarlyTerminateOnPrefix(searchSort, indexSort);
    +  }
    +
    +  private static boolean canEarlyTerminateOnDocId(Sort searchSort, Sort 
indexSort) {
    --- End diff --
    
    The early termination is not based on the `indexSort` so we should only 
check the `searchSort`. 


---

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

Reply via email to