Skip to content

[8.5] Ensure tasks with banned parents always get cancelled (#90188)#90224

Merged
elasticsearchmachine merged 2 commits intoelastic:8.5from
pxsalehi:backport/8.5/pr-90188
Sep 22, 2022
Merged

[8.5] Ensure tasks with banned parents always get cancelled (#90188)#90224
elasticsearchmachine merged 2 commits intoelastic:8.5from
pxsalehi:backport/8.5/pr-90188

Conversation

@pxsalehi
Copy link
Copy Markdown
Member

Backports the following commits to 8.5:

The check used to entirely skip parent lookup relies on
ConcurrentHashMap#isEmpty() which could return inconsistent results, and
potentially skip the cancellation of a task with a banned parent upon
registration, and it doesn't seem to have a benefit considering the hash
code computation.

Closes elastic#88201
@pxsalehi pxsalehi added :Distributed/Task Management Issues for anything around the Tasks API - both persistent and node level. >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Distributed Meta label for distributed team. labels Sep 22, 2022
@pxsalehi
Copy link
Copy Markdown
Member Author

@elasticmachine run elasticsearch-ci/part-2
(Known issue #90197)

@pxsalehi
Copy link
Copy Markdown
Member Author

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit ff55cd0 into elastic:8.5 Sep 22, 2022
@pxsalehi pxsalehi deleted the backport/8.5/pr-90188 branch September 22, 2022 12:32
@csoulios csoulios added v8.5.0 and removed v8.5.1 labels Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Distributed/Task Management Issues for anything around the Tasks API - both persistent and node level. Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v8.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants