mayya-sharipova commented on pull request #2075: URL: https://github.com/apache/lucene-solr/pull/2075#issuecomment-731377310
I've caught with @jimczi offline, and we have decided that as all index segments are sorted in the same way, `TopFieldCollector` should inform `FieldComparator` about index sort only once and not for every leaf. This change is reflected in 5aa08f51b. For 8.x branch `FieldComparator::usesIndexSort` method will be combined into [FieldComparator::setCanUsePoints](https://github.com/apache/lucene-solr/blob/branch_8x/lucene/core/src/java/org/apache/lucene/search/FieldComparator.java#L152) @jimczi This PR is ready for another round of review, please review when you have time ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
