Skip to content

Commit ab55833

Browse files
authored
Fix for TopNOperatorTests (#109937)
1 parent c6e21a9 commit ab55833

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,6 @@ tests:
7575
method: "testFetchAllEntities"
7676
- class: "org.elasticsearch.xpack.ml.integration.AutodetectMemoryLimitIT"
7777
issue: "https://github.com/elastic/elasticsearch/issues/109904"
78-
- class: "org.elasticsearch.compute.operator.topn.TopNOperatorTests"
79-
issue: "https://github.com/elastic/elasticsearch/issues/109915"
80-
method: "testRandomMultiValuesTopN"
8178

8279

8380
# Examples:

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
import static org.elasticsearch.test.ESTestCase.between;
1818
import static org.elasticsearch.test.ESTestCase.randomBoolean;
1919
import static org.elasticsearch.test.ESTestCase.randomDouble;
20+
import static org.elasticsearch.test.ESTestCase.randomFloat;
2021
import static org.elasticsearch.test.ESTestCase.randomInt;
2122
import static org.elasticsearch.test.ESTestCase.randomLong;
2223
import static org.elasticsearch.test.ESTestCase.randomRealisticUnicodeOfCodepointLengthBetween;
@@ -31,7 +32,7 @@ public static Object randomValue(ElementType e) {
3132
return switch (e) {
3233
case INT -> randomInt();
3334
case LONG -> randomLong();
34-
case FLOAT -> Float.intBitsToFloat(randomInt());
35+
case FLOAT -> randomFloat();
3536
case DOUBLE -> randomDouble();
3637
case BYTES_REF -> new BytesRef(randomRealisticUnicodeOfCodepointLengthBetween(0, 5)); // TODO: also test spatial WKB
3738
case BOOLEAN -> randomBoolean();

0 commit comments

Comments
 (0)