Skip to content

Commit 7e632b6

Browse files
authored
Extend timeout in testSearchContextIsCleanedUpAfterPageTimeoutForAggregationQueries (#132674)
Fix #132619 or try to, anyway. It looks like the cursor is sometimes already closed before we assert that it was created. Keep it alive a little longer (1s instead of 0.5s) so the assertion that the cursor is alive has a higher chance of succeeding.
1 parent 2417985 commit 7e632b6

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -486,9 +486,6 @@ tests:
486486
- class: org.elasticsearch.xpack.ml.integration.RevertModelSnapshotIT
487487
method: testRevertModelSnapshot_DeleteInterveningResults
488488
issue: https://github.com/elastic/elasticsearch/issues/132349
489-
- class: org.elasticsearch.xpack.sql.action.SqlSearchPageTimeoutIT
490-
method: testSearchContextIsCleanedUpAfterPageTimeoutForAggregationQueries
491-
issue: https://github.com/elastic/elasticsearch/issues/132619
492489
- class: org.elasticsearch.xpack.ml.integration.TextEmbeddingQueryIT
493490
method: testHybridSearch
494491
issue: https://github.com/elastic/elasticsearch/issues/132703

x-pack/plugin/sql/src/internalClusterTest/java/org/elasticsearch/xpack/sql/action/SqlSearchPageTimeoutIT.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public void testSearchContextIsCleanedUpAfterPageTimeout(String query) throws Ex
4747

4848
SqlQueryResponse response = new SqlQueryRequestBuilder(client()).query(query)
4949
.fetchSize(1)
50-
.pageTimeout(TimeValue.timeValueMillis(500))
50+
.pageTimeout(TimeValue.timeValueMillis(1000))
5151
.get();
5252

5353
assertTrue(response.hasCursor());

0 commit comments

Comments
 (0)