fencepost

romseygeek · romseygeek · commit cb9a7f54a15a · 2025-11-05T09:57:19.000Z
diff --git a/server/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/LongValuesComparatorSource.java b/server/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/LongValuesComparatorSource.java
@@ -228,7 +228,7 @@ public int nextDoc() throws IOException {
 
             @Override
             public int advance(int target) throws IOException {
-                if (target > maxDoc) {
+                if (target >= maxDoc) {
                     return doc = NO_MORE_DOCS;
                 }
                 // All documents are guaranteed to have a value, as all invocations of getLongValues

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ public int nextDoc() throws IOException {`
`228`	`228`
`229`	`229`	`@Override`
`230`	`230`	`public int advance(int target) throws IOException {`
`231`		`- if (target > maxDoc) {`
	`231`	`+ if (target >= maxDoc) {`
`232`	`232`	`return doc = NO_MORE_DOCS;`
`233`	`233`	`}`
`234`	`234`	`// All documents are guaranteed to have a value, as all invocations of getLongValues`