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 #17318

Merged
merged 6 commits into from
May 11, 2023

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:

@cla-bot cla-bot bot added the cla-signed label May 2, 2023
@losipiuk losipiuk changed the title Lo/spec spec flag in task status Add flag if task is speculative to TaskStatus May 2, 2023
@losipiuk losipiuk deleted the branch trinodb:master May 2, 2023 12:14
@losipiuk losipiuk closed this May 2, 2023
@losipiuk losipiuk reopened this May 2, 2023
@losipiuk losipiuk changed the base branch from lo/spec-spec-flag-in-task-status to master May 2, 2023 12:17
@losipiuk losipiuk requested review from arhimondr and findepi May 2, 2023 12:32
@losipiuk losipiuk force-pushed the lo/spec-spec-flag-in-task-status branch 3 times, most recently from 9fb0725 to 1cf45a5 Compare May 3, 2023 11:36
@losipiuk losipiuk force-pushed the lo/spec-spec-flag-in-task-status branch 2 times, most recently from dbf11d7 to d87c82e Compare May 9, 2023 17:04
@losipiuk losipiuk force-pushed the lo/spec-spec-flag-in-task-status branch from d87c82e to 309df3e Compare May 9, 2023 17:21
losipiuk added 2 commits May 10, 2023 20:47
Previously there was a brittle assumption that addOrUpdate is called
with just once for non-speculative task and for speculative task we get
exactly two calls
 - add with speculative priority and with
 - update with non-speculative priority

 This PR makes logic more explicit and relaxes assumptions
@losipiuk losipiuk force-pushed the lo/spec-spec-flag-in-task-status branch from 309df3e to 2c302ba Compare May 10, 2023 18:47
@losipiuk
Copy link
Member Author

CI: #17158

@losipiuk losipiuk merged commit f32875c into trinodb:master May 11, 2023
@github-actions github-actions bot added this to the 418 milestone May 11, 2023
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.

4 participants