Re: [PR] Check ahead if we can get the count [lucene]

jpountz commented on code in PR #13899: URL: https://github.com/apache/lucene/pull/13899#discussion_r1815247300 ## lucene/core/src/java/org/apache/lucene/search/IndexSortSortedNumericDocValuesRangeQuery.java: ## @@ -186,10 +186,44 @@ public boolean isCacheable(LeafReaderContext

Re: [PR] Check ahead if we can get the count [lucene]

LuXugang merged PR #13899: URL: https://github.com/apache/lucene/pull/13899 -- 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.ap

Re: [PR] Check ahead if we can get the count [lucene]

LuXugang commented on code in PR #13899: URL: https://github.com/apache/lucene/pull/13899#discussion_r1816108095 ## lucene/core/src/java/org/apache/lucene/search/IndexSortSortedNumericDocValuesRangeQuery.java: ## @@ -186,10 +186,44 @@ public boolean isCacheable(LeafReaderContext

Re: [PR] Check ahead if we can get the count [lucene]

LuXugang commented on PR #13899: URL: https://github.com/apache/lucene/pull/13899#issuecomment-2426154458 > The logic makes sense to me but it's a bit hard to read, could we avoid touching `getDocIdSetIteratorOrNull` and only have new logic in the `Weight#count` impl? Thank you for y

[PR] Check ahead if we can get the count [lucene]

LuXugang opened a new pull request, #13899: URL: https://github.com/apache/lucene/pull/13899 Check whether the IndexSortSortedNumericDocValuesRangeQuery's count can be obtained before traversing the BKD tree or performing binary search using DocValues. see https://github.com/apache/l