Skip to content

Commit 4254ca3

Browse files
authored
Kderusso/sparse vector ci failure (#117930) (#118014)
* Fix CI failure in SparseVectorQueryBuilderTests
1 parent 20957d6 commit 4254ca3

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -411,6 +411,3 @@ tests:
411411
issue: https://github.com/elastic/elasticsearch/issues/117805
412412
- class: org.elasticsearch.xpack.security.authc.ldap.UserAttributeGroupsResolverTests
413413
issue: https://github.com/elastic/elasticsearch/issues/116537
414-
- class: org.elasticsearch.xpack.core.ml.search.SparseVectorQueryBuilderTests
415-
method: testToQuery
416-
issue: https://github.com/elastic/elasticsearch/issues/117998

x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/ml/search/SparseVectorQueryBuilderTests.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -232,6 +232,12 @@ public void testToQuery() throws IOException {
232232

233233
private void testDoToQuery(SparseVectorQueryBuilder queryBuilder, SearchExecutionContext context) throws IOException {
234234
Query query = queryBuilder.doToQuery(context);
235+
236+
// test query builder can randomly have no vectors, which rewrites to a MatchNoneQuery - nothing more to do in this case.
237+
if (query instanceof MatchNoDocsQuery) {
238+
return;
239+
}
240+
235241
assertTrue(query instanceof SparseVectorQueryWrapper);
236242
var sparseQuery = (SparseVectorQueryWrapper) query;
237243
if (queryBuilder.shouldPruneTokens()) {

0 commit comments

Comments
 (0)