Skip to content

Commit ec78c0d

Browse files
committed
removing unnecessary listener
1 parent 78a4e06 commit ec78c0d

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

modules/data-streams/src/main/java/org/elasticsearch/datastreams/action/PostDataStreamTransportAction.java

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ public void onResponse(AcknowledgedResponse acknowledgedResponse) {
166166
CountDownActionListener settingCountDownListener = new CountDownActionListener(
167167
requestSettings.size() + 1,
168168
new ActionListener<>() {
169+
// Called once all settings are completed for all indices
169170
@Override
170171
public void onResponse(Void unused) {
171172
ComposableIndexTemplate effectiveIndexTemplate = clusterService.state()
@@ -194,36 +195,24 @@ public void onFailure(Exception e) {
194195
}
195196
);
196197
settingCountDownListener.onResponse(null); // handles the case when there were zero settings
197-
ActionListener<PostDataStreamAction.DataStreamResponse.IndexSettingResult> indexSettingResultListener =
198-
new ActionListener<>() {
199-
// Called each time we have results for all indices for a single setting
200-
@Override
201-
public void onResponse(PostDataStreamAction.DataStreamResponse.IndexSettingResult indexSettingResult) {
202-
indexSettingResults.add(indexSettingResult);
203-
settingCountDownListener.onResponse(null);
204-
}
205-
206-
@Override
207-
public void onFailure(Exception e) {
208-
settingCountDownListener.onFailure(e);
209-
}
210-
};
211198
for (String setting : requestSettings.keySet()) {
212199
if (APPLY_TO_BACKING_INDICES.contains(setting)) {
213200
final List<PostDataStreamAction.DataStreamResponse.IndexSettingError> errors = new ArrayList<>();
214201
CountDownActionListener indexCountDownListener = new CountDownActionListener(
215202
concreteIndices.size() + 1,
216203
new ActionListener<>() {
204+
// Called when all indices for a single setting are complete
217205
@Override
218206
public void onResponse(Void unused) {
219-
indexSettingResultListener.onResponse(
207+
indexSettingResults.add(
220208
new PostDataStreamAction.DataStreamResponse.IndexSettingResult(setting, true, errors)
221209
);
210+
settingCountDownListener.onResponse(null);
222211
}
223212

224213
@Override
225214
public void onFailure(Exception e) {
226-
indexSettingResultListener.onFailure(e);
215+
settingCountDownListener.onFailure(e);
227216
}
228217
}
229218
);
@@ -236,6 +225,7 @@ public void onFailure(Exception e) {
236225
masterNodeTimeout,
237226
ackTimeout,
238227
new ActionListener<>() {
228+
// Called when a single setting for a single index is complete
239229
@Override
240230
public void onResponse(AcknowledgedResponse response) {
241231
if (response.isAcknowledged() == false) {
@@ -247,7 +237,6 @@ public void onResponse(AcknowledgedResponse response) {
247237
);
248238
}
249239
indexCountDownListener.onResponse(null);
250-
251240
}
252241

253242
@Override

0 commit comments

Comments
 (0)