Skip to content

Commit 21bdf4d

Browse files
authored
Fix DocValuesCodecDuelTests testDuel (#112084) (#112113)
We need to check the returned doc id from advance() before accessing the values of the current document. Closes #112082
1 parent 91035f8 commit 21bdf4d

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

server/src/test/java/org/elasticsearch/index/codec/tsdb/DocValuesCodecDuelTests.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,9 @@ private void assertSortedDocValues(LeafReader baselineReader, LeafReader contend
141141
for (int i = 0; i < docIdsToAdvanceTo.length; i++) {
142142
int docId = docIdsToAdvanceTo[i];
143143
int baselineTarget = assertAdvance(docId, baselineReader, contenderReader, baseline, contender);
144+
if (baselineTarget == NO_MORE_DOCS) {
145+
break;
146+
}
144147
assertEquals(baseline.ordValue(), contender.ordValue());
145148
assertEquals(baseline.lookupOrd(baseline.ordValue()), contender.lookupOrd(contender.ordValue()));
146149
i = shouldSkipDocIds(i, docId, baselineTarget, docIdsToAdvanceTo);

0 commit comments

Comments
 (0)