antonha commented on code in PR #13149: URL: https://github.com/apache/lucene/pull/13149#discussion_r1528679700
########## lucene/core/src/java/org/apache/lucene/search/PointRangeQuery.java: ########## @@ -185,6 +186,13 @@ public void visit(DocIdSetIterator iterator) throws IOException { adder.add(iterator); } + @Override + public void visit(IntsRef ref) { + for (int i = ref.offset; i < ref.offset + ref.length; i++) { Review Comment: That would probably be better. I don't think we should expect huge performance increases from it (since it would only affect narrow range queries). But it would make a large difference if a third implementation of the adder is ever used. I have limited time this week to look at it - I will try to find some time for it though! -- 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