Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

React more promptly to task cancellation while waiting for the cluster to unblock #117971

Open
nielsbauman opened this issue Dec 4, 2024 · 1 comment
Open

React more promptly to task cancellation while waiting for the cluster to unblock #117971

nielsbauman opened this issue Dec 4, 2024 · 1 comment
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >enhancement Team:Distributed Indexing Meta label for Distributed Indexing team

Comments

@nielsbauman
Copy link
Contributor

Both in TransportLocalClusterStateAction and TransportMasterNodeAction we wait for the cluster to unblock. Currently, if a task gets cancelled while waiting, we don't send a response until the cluster gets unblocked (which could be arbitrarily far in the future) or until the timeout occurs. We should make use of CancellableTask#addListener to react more promptly to task cancellation.

Follow-up from #117230 (comment)
@nielsbauman nielsbauman added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >enhancement Team:Distributed Indexing Meta label for Distributed Indexing team labels Dec 4, 2024
@nielsbauman nielsbauman mentioned this issue Dec 4, 2024
Merged
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >enhancement Team:Distributed Indexing Meta label for Distributed Indexing team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nielsbauman @elasticsearchmachine