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.

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

Add flag if task is speculative to TaskStatus #17316

Closed
wants to merge 5 commits into from

Conversation

losipiuk
Copy link
Member

@losipiuk losipiuk commented May 2, 2023

On top of #17305

Release notes

(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

losipiuk added 5 commits May 1, 2023 13:17
It was possible that node lease could not be released for
failed query is failure happened just after node was allocated
but before task obtained task instance handle.

The cleanup routine which iterated over nodeAcquisitions map
did not work properly then, as entry was already removed from the map.
@cla-bot cla-bot bot added the cla-signed label May 2, 2023
@losipiuk losipiuk requested a review from arhimondr May 2, 2023 10:41
@losipiuk
Copy link
Member Author

losipiuk commented May 2, 2023

wrong origin. Replaced with #17318

@losipiuk losipiuk closed this May 2, 2023
@losipiuk losipiuk deleted the lo/spec-spec-flag-in-task-status branch May 2, 2023 12:14
@losipiuk losipiuk restored the lo/spec-spec-flag-in-task-status branch May 2, 2023 12:14
@losipiuk losipiuk deleted the lo/spec-spec-flag-in-task-status branch May 2, 2023 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant