Skip to content

Commit 92eb324

Browse files
authored
REST high-level Client: remove deprecated API methods (#31200)
This commit removes all the API methods that accept a `Header` varargs argument, in favour of the newly introduced API methods that accept a `RequestOptions` argument. Relates to #31069
1 parent bd81b95 commit 92eb324

18 files changed

+128
-1056
lines changed

client/rest-high-level/src/main/java/org/elasticsearch/client/ClusterClient.java

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919

2020
package org.elasticsearch.client;
2121

22-
import org.apache.http.Header;
2322
import org.elasticsearch.action.ActionListener;
2423
import org.elasticsearch.action.admin.cluster.health.ClusterHealthRequest;
2524
import org.elasticsearch.action.admin.cluster.health.ClusterHealthResponse;
@@ -59,20 +58,6 @@ public ClusterUpdateSettingsResponse putSettings(ClusterUpdateSettingsRequest cl
5958
options, ClusterUpdateSettingsResponse::fromXContent, emptySet());
6059
}
6160

62-
/**
63-
* Updates cluster wide specific settings using the Cluster Update Settings API.
64-
* <p>
65-
* See <a href="https://www.elastic.co/guide/en/elasticsearch/reference/current/cluster-update-settings.html"> Cluster Update Settings
66-
* API on elastic.co</a>
67-
* @deprecated Prefer {@link #putSettings(ClusterUpdateSettingsRequest, RequestOptions)}
68-
*/
69-
@Deprecated
70-
public ClusterUpdateSettingsResponse putSettings(ClusterUpdateSettingsRequest clusterUpdateSettingsRequest, Header... headers)
71-
throws IOException {
72-
return restHighLevelClient.performRequestAndParseEntity(clusterUpdateSettingsRequest, RequestConverters::clusterPutSettings,
73-
ClusterUpdateSettingsResponse::fromXContent, emptySet(), headers);
74-
}
75-
7661
/**
7762
* Asynchronously updates cluster wide specific settings using the Cluster Update Settings API.
7863
* See <a href="https://www.elastic.co/guide/en/elasticsearch/reference/current/cluster-update-settings.html"> Cluster Update Settings
@@ -86,19 +71,6 @@ public void putSettingsAsync(ClusterUpdateSettingsRequest clusterUpdateSettingsR
8671
restHighLevelClient.performRequestAsyncAndParseEntity(clusterUpdateSettingsRequest, RequestConverters::clusterPutSettings,
8772
options, ClusterUpdateSettingsResponse::fromXContent, listener, emptySet());
8873
}
89-
/**
90-
* Asynchronously updates cluster wide specific settings using the Cluster Update Settings API.
91-
* <p>
92-
* See <a href="https://www.elastic.co/guide/en/elasticsearch/reference/current/cluster-update-settings.html"> Cluster Update Settings
93-
* API on elastic.co</a>
94-
* @deprecated Prefer {@link #putSettingsAsync(ClusterUpdateSettingsRequest, RequestOptions, ActionListener)}
95-
*/
96-
@Deprecated
97-
public void putSettingsAsync(ClusterUpdateSettingsRequest clusterUpdateSettingsRequest,
98-
ActionListener<ClusterUpdateSettingsResponse> listener, Header... headers) {
99-
restHighLevelClient.performRequestAsyncAndParseEntity(clusterUpdateSettingsRequest, RequestConverters::clusterPutSettings,
100-
ClusterUpdateSettingsResponse::fromXContent, listener, emptySet(), headers);
101-
}
10274

10375
/**
10476
* Get cluster health using the Cluster Health API.

client/rest-high-level/src/main/java/org/elasticsearch/client/IndicesClient.java

Lines changed: 0 additions & 430 deletions
Large diffs are not rendered by default.

client/rest-high-level/src/main/java/org/elasticsearch/client/RestHighLevelClient.java

Lines changed: 0 additions & 344 deletions
Large diffs are not rendered by default.

client/rest-high-level/src/main/java/org/elasticsearch/client/TasksClient.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ public CancelTasksResponse cancel(CancelTasksRequest cancelTasksRequest, Request
8484
cancelTasksRequest,
8585
RequestConverters::cancelTasks,
8686
options,
87-
parser -> CancelTasksResponse.fromXContent(parser),
87+
CancelTasksResponse::fromXContent,
8888
emptySet()
8989
);
9090
}
@@ -103,7 +103,7 @@ public void cancelAsync(CancelTasksRequest cancelTasksRequest, RequestOptions op
103103
cancelTasksRequest,
104104
RequestConverters::cancelTasks,
105105
options,
106-
parser -> CancelTasksResponse.fromXContent(parser),
106+
CancelTasksResponse::fromXContent,
107107
listener,
108108
emptySet()
109109
);

client/rest-high-level/src/test/java/org/elasticsearch/client/BulkProcessorIT.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,8 @@
5656
public class BulkProcessorIT extends ESRestHighLevelClientTestCase {
5757

5858
private static BulkProcessor.Builder initBulkProcessorBuilder(BulkProcessor.Listener listener) {
59-
return BulkProcessor.builder(highLevelClient()::bulkAsync, listener);
59+
return BulkProcessor.builder(
60+
(request, bulkListener) -> highLevelClient().bulkAsync(request, RequestOptions.DEFAULT, bulkListener), listener);
6061
}
6162

6263
public void testThatBulkProcessorCountIsCorrect() throws Exception {

client/rest-high-level/src/test/java/org/elasticsearch/client/BulkProcessorRetryIT.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,8 @@ public class BulkProcessorRetryIT extends ESRestHighLevelClientTestCase {
4848
private static final String TYPE_NAME = "type";
4949

5050
private static BulkProcessor.Builder initBulkProcessorBuilder(BulkProcessor.Listener listener) {
51-
return BulkProcessor.builder(highLevelClient()::bulkAsync, listener);
51+
return BulkProcessor.builder(
52+
(request, bulkListener) -> highLevelClient().bulkAsync(request, RequestOptions.DEFAULT, bulkListener), listener);
5253
}
5354

5455
public void testBulkRejectionLoadWithoutBackoff() throws Exception {

client/rest-high-level/src/test/java/org/elasticsearch/client/ClusterClientIT.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,6 @@ public void testClusterPutSettings() throws IOException {
6363
setRequest.persistentSettings(map);
6464

6565
ClusterUpdateSettingsResponse setResponse = execute(setRequest, highLevelClient().cluster()::putSettings,
66-
highLevelClient().cluster()::putSettingsAsync, highLevelClient().cluster()::putSettings,
6766
highLevelClient().cluster()::putSettingsAsync);
6867

6968
assertAcked(setResponse);
@@ -86,7 +85,6 @@ public void testClusterPutSettings() throws IOException {
8685
resetRequest.persistentSettings("{\"" + persistentSettingKey + "\": null }", XContentType.JSON);
8786

8887
ClusterUpdateSettingsResponse resetResponse = execute(resetRequest, highLevelClient().cluster()::putSettings,
89-
highLevelClient().cluster()::putSettingsAsync, highLevelClient().cluster()::putSettings,
9088
highLevelClient().cluster()::putSettingsAsync);
9189

9290
assertThat(resetResponse.getTransientSettings().get(transientSettingKey), equalTo(null));
@@ -108,7 +106,6 @@ public void testClusterUpdateSettingNonExistent() {
108106
clusterUpdateSettingsRequest.transientSettings(Settings.builder().put(setting, value).build());
109107

110108
ElasticsearchException exception = expectThrows(ElasticsearchException.class, () -> execute(clusterUpdateSettingsRequest,
111-
highLevelClient().cluster()::putSettings, highLevelClient().cluster()::putSettingsAsync,
112109
highLevelClient().cluster()::putSettings, highLevelClient().cluster()::putSettingsAsync));
113110
assertThat(exception.status(), equalTo(RestStatus.BAD_REQUEST));
114111
assertThat(exception.getMessage(), equalTo(

0 commit comments

Comments
 (0)