Skip to content

Commit 8761067

Browse files
authored
Fix + unmute SampleTests (#127959)
1 parent 7872e84 commit 8761067

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

muted-tests.yml

-6
Original file line numberDiff line numberDiff line change
@@ -462,12 +462,6 @@ tests:
462462
- class: org.elasticsearch.indices.stats.IndexStatsIT
463463
method: testThrottleStats
464464
issue: https://github.com/elastic/elasticsearch/issues/126359
465-
- class: org.elasticsearch.xpack.esql.expression.function.aggregate.SampleTests
466-
method: testGroupingAggregate {TestCase=<long unicode KEYWORDs>}
467-
issue: https://github.com/elastic/elasticsearch/issues/127950
468-
- class: org.elasticsearch.xpack.esql.expression.function.aggregate.SampleTests
469-
method: testGroupingAggregate {TestCase=<long unicode TEXTs>}
470-
issue: https://github.com/elastic/elasticsearch/issues/127951
471465
- class: org.elasticsearch.search.vectors.IVFKnnFloatVectorQueryTests
472466
method: testRandomWithFilter
473467
issue: https://github.com/elastic/elasticsearch/issues/127963

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/aggregate/SampleTests.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,12 @@ public static Iterable<Object[]> parameters() {
5050
MultiRowTestCaseSupplier.booleanCases(1, 1000),
5151
MultiRowTestCaseSupplier.ipCases(1, 1000),
5252
MultiRowTestCaseSupplier.versionCases(1, 1000),
53-
MultiRowTestCaseSupplier.stringCases(1, 1000, DataType.KEYWORD),
54-
MultiRowTestCaseSupplier.stringCases(1, 1000, DataType.TEXT),
53+
MultiRowTestCaseSupplier.stringCases(1, 20, DataType.KEYWORD),
54+
MultiRowTestCaseSupplier.stringCases(1, 20, DataType.TEXT),
5555
MultiRowTestCaseSupplier.geoPointCases(1, 1000, MultiRowTestCaseSupplier.IncludingAltitude.NO),
5656
MultiRowTestCaseSupplier.cartesianPointCases(1, 1000, MultiRowTestCaseSupplier.IncludingAltitude.NO),
57-
MultiRowTestCaseSupplier.geoShapeCasesWithoutCircle(1, 100, MultiRowTestCaseSupplier.IncludingAltitude.NO),
58-
MultiRowTestCaseSupplier.cartesianShapeCasesWithoutCircle(1, 100, MultiRowTestCaseSupplier.IncludingAltitude.NO)
57+
MultiRowTestCaseSupplier.geoShapeCasesWithoutCircle(1, 20, MultiRowTestCaseSupplier.IncludingAltitude.NO),
58+
MultiRowTestCaseSupplier.cartesianShapeCasesWithoutCircle(1, 20, MultiRowTestCaseSupplier.IncludingAltitude.NO)
5959
)
6060
.flatMap(List::stream)
6161
.map(fieldCaseSupplier -> makeSupplier(fieldCaseSupplier, limitCaseSupplier))

0 commit comments

Comments
 (0)