mkhludnev commented on a change in pull request #1024: LUCENE-9050: MultiTermIntervalsSource should call visitLeaf() in visit URL: https://github.com/apache/lucene-solr/pull/1024#discussion_r349051139
########## File path: lucene/queries/src/java/org/apache/lucene/queries/intervals/MultiTermIntervalsSource.java ########## @@ -96,7 +96,7 @@ public MatchesIterator matches(String field, LeafReaderContext ctx, int doc) thr @Override public void visit(String field, QueryVisitor visitor) { - + visitor.visitLeaf(new IntervalQuery(field, this)); Review comment: Isn't it better to stub AtomatonQuery and yield it here. IIRC it resolves simplest MTQ highlighting cases as nobrainer. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org