From efba3ad863cdb8900eeab427747b2f836ed45dc0 Mon Sep 17 00:00:00 2001 From: Benjamin Trent <4357155+benwtrent@users.noreply.github.com> Date: Fri, 10 Jan 2025 08:57:23 -0500 Subject: [PATCH] Fix knn query tests as we now do queries with k --- muted-tests.yml | 6 ------ .../test/search.vectors/110_knn_query_with_filter.yml | 4 ++-- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/muted-tests.yml b/muted-tests.yml index 64a890f5e7298..8b8332140d517 100644 --- a/muted-tests.yml +++ b/muted-tests.yml @@ -428,12 +428,6 @@ tests: - class: org.elasticsearch.upgrades.SearchStatesIT method: testCanMatch issue: https://github.com/elastic/elasticsearch/issues/118718 -- class: org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT - method: "test {p0=search.vectors/110_knn_query_with_filter/PRE_FILTER: knn query with internal filter as pre-filter}" - issue: https://github.com/elastic/elasticsearch/issues/119804 -- class: org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT - method: "test {p0=search.vectors/110_knn_query_with_filter/PRE_FILTER: knn query with alias filter as pre-filter}" - issue: https://github.com/elastic/elasticsearch/issues/119805 - class: org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT issue: https://github.com/elastic/elasticsearch/issues/119806 - class: org.elasticsearch.search.profile.dfs.DfsProfilerIT diff --git a/rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/search.vectors/110_knn_query_with_filter.yml b/rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/search.vectors/110_knn_query_with_filter.yml index 3d4841a16d82d..c69f067904e2c 100644 --- a/rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/search.vectors/110_knn_query_with_filter.yml +++ b/rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/search.vectors/110_knn_query_with_filter.yml @@ -1,8 +1,8 @@ # test how knn query interacts with filters setup: - requires: - cluster_features: "gte_v8.12.0" - reason: 'knn as query added in 8.12' + cluster_features: "search.vectors.k_param_supported" + reason: 'k param for knn as query is required' - do: indices.create: