zhaih commented on code in PR #11840: URL: https://github.com/apache/lucene/pull/11840#discussion_r995390126
########## lucene/core/src/java/org/apache/lucene/document/FeatureQuery.java: ########## @@ -50,12 +49,12 @@ final class FeatureQuery extends Query { } @Override - public Query rewrite(IndexReader reader) throws IOException { - FeatureFunction rewritten = function.rewrite(reader); + public Query rewrite(IndexSearcher indexSearcher) throws IOException { + FeatureFunction rewritten = function.rewrite(indexSearcher.getIndexReader()); Review Comment: Sure, I'll also change the `MultiTermQuery.rewriteMethod`, was planning to do them in another PR, but seems not too bad to merge all into this one. -- 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