From f7f9878712f7daf5bc1d49ca090f21f14e5f5f40 Mon Sep 17 00:00:00 2001 From: Jan Kuipers Date: Fri, 9 May 2025 10:47:31 +0200 Subject: [PATCH] Fix + unmute SampleTests --- muted-tests.yml | 6 ------ .../esql/expression/function/aggregate/SampleTests.java | 8 ++++---- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/muted-tests.yml b/muted-tests.yml index 317f0dbe6abc4..a76a168e8af54 100644 --- a/muted-tests.yml +++ b/muted-tests.yml @@ -462,12 +462,6 @@ tests: - class: org.elasticsearch.indices.stats.IndexStatsIT method: testThrottleStats issue: https://github.com/elastic/elasticsearch/issues/126359 -- class: org.elasticsearch.xpack.esql.expression.function.aggregate.SampleTests - method: testGroupingAggregate {TestCase=} - issue: https://github.com/elastic/elasticsearch/issues/127950 -- class: org.elasticsearch.xpack.esql.expression.function.aggregate.SampleTests - method: testGroupingAggregate {TestCase=} - issue: https://github.com/elastic/elasticsearch/issues/127951 # Examples: # diff --git a/x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/aggregate/SampleTests.java b/x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/aggregate/SampleTests.java index 61ad7345a4a7b..8249532d5bcc9 100644 --- a/x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/aggregate/SampleTests.java +++ b/x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/expression/function/aggregate/SampleTests.java @@ -50,12 +50,12 @@ public static Iterable parameters() { MultiRowTestCaseSupplier.booleanCases(1, 1000), MultiRowTestCaseSupplier.ipCases(1, 1000), MultiRowTestCaseSupplier.versionCases(1, 1000), - MultiRowTestCaseSupplier.stringCases(1, 1000, DataType.KEYWORD), - MultiRowTestCaseSupplier.stringCases(1, 1000, DataType.TEXT), + MultiRowTestCaseSupplier.stringCases(1, 20, DataType.KEYWORD), + MultiRowTestCaseSupplier.stringCases(1, 20, DataType.TEXT), MultiRowTestCaseSupplier.geoPointCases(1, 1000, MultiRowTestCaseSupplier.IncludingAltitude.NO), MultiRowTestCaseSupplier.cartesianPointCases(1, 1000, MultiRowTestCaseSupplier.IncludingAltitude.NO), - MultiRowTestCaseSupplier.geoShapeCasesWithoutCircle(1, 100, MultiRowTestCaseSupplier.IncludingAltitude.NO), - MultiRowTestCaseSupplier.cartesianShapeCasesWithoutCircle(1, 100, MultiRowTestCaseSupplier.IncludingAltitude.NO) + MultiRowTestCaseSupplier.geoShapeCasesWithoutCircle(1, 20, MultiRowTestCaseSupplier.IncludingAltitude.NO), + MultiRowTestCaseSupplier.cartesianShapeCasesWithoutCircle(1, 20, MultiRowTestCaseSupplier.IncludingAltitude.NO) ) .flatMap(List::stream) .map(fieldCaseSupplier -> makeSupplier(fieldCaseSupplier, limitCaseSupplier))