bugmakerrrrrr commented on PR #14418: URL: https://github.com/apache/lucene/pull/14418#issuecomment-2763256416
> Can you help me understand what work this change helps save? Thanks for your reply. I think if the filter query matches no docs, the knn query must also match no docs. Before this small change, we always executed search tasks for each leaf reader even if the filter query is `MatchNoDocsQuery`. https://github.com/apache/lucene/blob/a7ff87fdb5479a2f1e89746d88c80023b0aecbb1/lucene/core/src/java/org/apache/lucene/search/AbstractKnnVectorQuery.java#L89-L98 -- 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
