-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
fix: update the execution sweep sql to resolve the exec deletion conflict #20603
Conversation
…flict Resolve the execution deleteion conflict when there are tasks still referenced by the execution, remained execs can wait for next sweep cycle. Signed-off-by: chlins <chlins.zhang@gmail.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #20603 +/- ##
==========================================
- Coverage 67.56% 66.24% -1.32%
==========================================
Files 991 1045 +54
Lines 109181 113476 +4295
Branches 2719 2845 +126
==========================================
+ Hits 73768 75177 +1409
- Misses 31449 34191 +2742
- Partials 3964 4108 +144
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
LGTM
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.
lgtm
Is there any chance this gets cherry-picked to 2.11? |
Signed-off-by: wang yan <wangyan@vmware.com>
Resolve the execution deletion conflict when there are tasks still referenced by the execution, remained execs can wait for next sweep cycle.
Comprehensive Summary of your change
Issue being fixed
Fixes
Please indicate you've done the following: