Skip to content

Commit 33620d0

Browse files
Address feedback: make capabilities intead of feature
1 parent 85063e1 commit 33620d0

File tree

8 files changed

+34
-38
lines changed

8 files changed

+34
-38
lines changed

modules/analysis-common/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ esplugin {
2020

2121
restResources {
2222
restApi {
23-
include '_common', 'indices', 'index', 'cluster', 'search', 'nodes', 'bulk', 'termvectors', 'explain', 'count'
23+
include '_common', 'indices', 'index', 'cluster', 'search', 'nodes', 'bulk', 'termvectors', 'explain', 'count', 'capabilities'
2424
}
2525
}
2626

modules/analysis-common/src/yamlRestTest/resources/rest-api-spec/test/indices.analyze/15_analyze.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,12 @@
6363
---
6464
"Custom analyzer is not buildable":
6565
- requires:
66-
cluster_features: [ "wrong_custom_analyzer_return_400" ]
67-
reason: "Returning 400 for wrong custom analyzer was added in this version"
66+
test_runner_features: [ capabilities ]
67+
reason: This capability required to run test
68+
capabilities:
69+
- method: GET
70+
path: /_analyze
71+
capabilities: [ wrong_custom_analyzer_returns_400 ]
6872

6973
- do:
7074
catch: bad_request

server/src/main/java/module-info.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -426,7 +426,6 @@
426426
provides org.elasticsearch.features.FeatureSpecification
427427
with
428428
org.elasticsearch.action.bulk.BulkFeatures,
429-
org.elasticsearch.action.admin.ActionFeatures,
430429
org.elasticsearch.features.FeatureInfrastructureFeatures,
431430
org.elasticsearch.rest.action.admin.cluster.ClusterRerouteFeatures,
432431
org.elasticsearch.index.mapper.MapperFeatures,
@@ -477,5 +476,4 @@
477476
exports org.elasticsearch.monitor.metrics;
478477
exports org.elasticsearch.plugins.internal.rewriter to org.elasticsearch.inference;
479478
exports org.elasticsearch.lucene.util.automaton;
480-
exports org.elasticsearch.action.admin;
481479
}

server/src/main/java/org/elasticsearch/action/admin/ActionFeatures.java

Lines changed: 0 additions & 30 deletions
This file was deleted.
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
/*
2+
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
3+
* or more contributor license agreements. Licensed under the "Elastic License
4+
* 2.0", the "GNU Affero General Public License v3.0 only", and the "Server Side
5+
* Public License v 1"; you may not use this file except in compliance with, at
6+
* your election, the "Elastic License 2.0", the "GNU Affero General Public
7+
* License v3.0 only", or the "Server Side Public License, v 1".
8+
*/
9+
10+
package org.elasticsearch.action.admin.indices.analyze;
11+
12+
import java.util.Set;
13+
14+
public final class AnalyzeCapabilities {
15+
private AnalyzeCapabilities() {}
16+
17+
private static final String WRONG_CUSTOM_ANALYZER_RETURNS_400_CAPABILITY = "wrong_custom_analyzer_returns_400";
18+
19+
public static final Set<String> CAPABILITIES = Set.of(WRONG_CUSTOM_ANALYZER_RETURNS_400_CAPABILITY);
20+
}

server/src/main/java/org/elasticsearch/action/admin/indices/analyze/TransportAnalyzeAction.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,8 +66,6 @@
6666
*/
6767
public class TransportAnalyzeAction extends TransportSingleShardAction<AnalyzeAction.Request, AnalyzeAction.Response> {
6868

69-
public static final NodeFeature WRONG_CUSTOM_ANALYZER_RETURN_400 = new NodeFeature("wrong_custom_analyzer_return_400");
70-
7169
private final Settings settings;
7270
private final IndicesService indicesService;
7371

server/src/main/java/org/elasticsearch/rest/action/admin/indices/RestAnalyzeAction.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
package org.elasticsearch.rest.action.admin.indices;
1010

1111
import org.elasticsearch.action.admin.indices.analyze.AnalyzeAction;
12+
import org.elasticsearch.action.admin.indices.analyze.AnalyzeCapabilities;
1213
import org.elasticsearch.client.internal.node.NodeClient;
1314
import org.elasticsearch.rest.BaseRestHandler;
1415
import org.elasticsearch.rest.RestRequest;
@@ -19,6 +20,7 @@
1920

2021
import java.io.IOException;
2122
import java.util.List;
23+
import java.util.Set;
2224

2325
import static org.elasticsearch.rest.RestRequest.Method.GET;
2426
import static org.elasticsearch.rest.RestRequest.Method.POST;
@@ -49,4 +51,9 @@ public RestChannelConsumer prepareRequest(final RestRequest request, final NodeC
4951
}
5052
}
5153

54+
@Override
55+
public Set<String> supportedCapabilities() {
56+
return AnalyzeCapabilities.CAPABILITIES;
57+
}
58+
5259
}

server/src/main/resources/META-INF/services/org.elasticsearch.features.FeatureSpecification

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
#
99

1010
org.elasticsearch.action.bulk.BulkFeatures
11-
org.elasticsearch.action.admin.ActionFeatures
1211
org.elasticsearch.features.FeatureInfrastructureFeatures
1312
org.elasticsearch.rest.action.admin.cluster.ClusterRerouteFeatures
1413
org.elasticsearch.index.IndexFeatures

0 commit comments

Comments
 (0)