bugmakerrrrrr commented on code in PR #15083:
URL: https://github.com/apache/lucene/pull/15083#discussion_r2635111352


##########
lucene/core/src/java/org/apache/lucene/index/DocValuesSkipper.java:
##########
@@ -122,4 +123,67 @@ public final void advance(long minValue, long maxValue) 
throws IOException {
       advance(maxDocID + 1);
     }
   }
+
+  /**
+   * Returns the minimum value for a field across all segments, or {@link 
Long#MIN_VALUE} if not
+   * available
+   *
+   * @param searcher a searcher over the index
+   * @param field the field to retrieve values for
+   */
+  public static long globalMinValue(IndexSearcher searcher, String field) 
throws IOException {

Review Comment:
   hi @romseygeek , I'm wondering if we can change the parameter of the global* 
method from `IndexSearcher ` to `IndexReader`, similar to the stats method in 
`PointValues`. I think this would make it more convenient for upper-level 
applications to use.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to