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

Reply via email to