Skip to content

Commit

Permalink
Fix issue with releasing resources in bulk tests (elastic#114186)
Browse files Browse the repository at this point in the history
A recent commit incidentally changed a release resources call from
doBefore to doAfter. Several tests depending on resources being released
synchronously which requires doBefore.

Closes elastic#114181
Closes elastic#114182
  • Loading branch information
Tim-Brooks committed Oct 7, 2024
1 parent 20a34f4 commit 27a3406
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ public void lastItems(List<DocWriteRequest<?>> items, Releasable releasable, Act
releasables.clear();
// We do not need to set this back to false as this will be the last request.
bulkInProgress = true;
client.bulk(bulkRequest, ActionListener.runAfter(new ActionListener<>() {
client.bulk(bulkRequest, ActionListener.runBefore(new ActionListener<>() {

private final boolean isFirstRequest = incrementalRequestSubmitted == false;

Expand Down

0 comments on commit 27a3406

Please sign in to comment.