Skip to content

Commit

Permalink
Fix assumption in TimeSeriesIT (elastic#106448)
Browse files Browse the repository at this point in the history
This should be an assumption instead of an assertion.

Closes elastic#106444
  • Loading branch information
dnhatn authored Mar 19, 2024
1 parent 823f174 commit 8357d2b
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,12 @@ public class TimeSeriesIT extends AbstractEsqlIntegTestCase {

@Override
protected EsqlQueryResponse run(EsqlQueryRequest request) {
assertTrue("timseries requires pragmas", canUseQueryPragmas());
assumeTrue("timseries requires pragmas", canUseQueryPragmas());
var settings = Settings.builder().put(request.pragmas().getSettings()).put(QueryPragmas.TIME_SERIES_MODE.getKey(), "true").build();
request.pragmas(new QueryPragmas(settings));
return super.run(request);
}

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/106444")
public void testEmpty() {
Settings settings = Settings.builder().put("mode", "time_series").putList("routing_path", List.of("pod")).build();
client().admin()
Expand Down

0 comments on commit 8357d2b

Please sign in to comment.