Skip to content

Commit ad65b40

Browse files
authored
Backport BlockMultiValuedTests fix (elastic#124766)
1 parent b0f8c19 commit ad65b40

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

x-pack/plugin/esql/compute/src/test/java/org/elasticsearch/compute/data/BlockMultiValuedTests.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
import java.util.ArrayList;
3030
import java.util.Arrays;
3131
import java.util.List;
32+
import java.util.Objects;
3233
import java.util.function.IntUnaryOperator;
3334
import java.util.stream.IntStream;
3435

@@ -133,10 +134,12 @@ public void testToMask() {
133134
if (elementType != ElementType.BOOLEAN) {
134135
return;
135136
}
136-
int positionCount = randomIntBetween(1, 16 * 1024);
137+
int positionCount = randomIntBetween(0, 16 * 1024);
137138
var b = RandomBlock.randomBlock(blockFactory(), elementType, positionCount, nullAllowed, 2, 10, 0, 0);
138139
try (ToMask mask = ((BooleanBlock) b.block()).toMask()) {
139-
assertThat(mask.hadMultivaluedFields(), equalTo(true));
140+
var anyNonNullValue = b.values().stream().anyMatch(Objects::nonNull);
141+
assertThat(mask.hadMultivaluedFields(), equalTo(anyNonNullValue));
142+
140143
for (int p = 0; p < b.values().size(); p++) {
141144
List<Object> v = b.values().get(p);
142145
if (v == null) {

0 commit comments

Comments
 (0)