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
Fixes #21716 - Fix cancelling when using bulk actions with polling #259
Conversation
f47fc3d
to
f8e6012
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not tested yet, but looks promissing, some suggestions insight + rebase needed
end | ||
|
||
def remaining_count | ||
total_count - output.fetch(:cancelled_count, 0) - output[:planned_count] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about initiating the value in initiate
so that we don't need to do fetch here? There is also the run_progress
method that this would be benefitial to have.
f8e6012
to
f810b89
Compare
Fixed and rebased |
Ruboocop is not happy |
Rubocop is finally happy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reading the code again, I think the issue we were talking about with potential issues of PlanNextBatch being in queue when cancelling, is actually handled here.
Thanks @adamruzicka |
No description provided.