romseygeek commented on PR #14759: URL: https://github.com/apache/lucene/pull/14759#issuecomment-2955094547
QueryVisitor is supposed to be a query introspection API, so I wouldn't change its behaviour here. If the problem is that we want to be able to query multiple ranges without creating giant boolean queries, then I'd suggest instead looking at adding a `newSetRangeQuery` method to `LongField`, and maybe promoting `LongMultiRangeBuilder` from the sandbox? -- 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
