-
Notifications
You must be signed in to change notification settings - Fork 5.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
gc_worker: resolve locks on offline stores #18383
Conversation
Signed-off-by: youjiali1995 <zlwgx1023@gmail.com>
BTW, I think we need cherry-pick it to release-3.0/3.1 too because offline stores are skipped during destroying ranges. It's possible an offline store becomes up if someone deletes a wrong store and reverts the operation. @MyonKeminta |
Codecov Report
@@ Coverage Diff @@
## master #18383 +/- ##
===========================================
Coverage 79.4581% 79.4581%
===========================================
Files 541 541
Lines 146457 146457
===========================================
Hits 116372 116372
Misses 20787 20787
Partials 9298 9298 |
Signed-off-by: youjiali1995 <zlwgx1023@gmail.com>
…95/tidb into gc/scan-offline-store
@MyonKeminta PTAL |
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
@@ -459,7 +459,7 @@ func (w *GCWorker) checkGCInterval(now time.Time) (bool, error) { | |||
} | |||
|
|||
// validateGCLiftTime checks whether life time is small than min gc life time. | |||
func (w *GCWorker) validateGCLiftTime(lifeTime time.Duration) (time.Duration, error) { |
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.
Oh
Signed-off-by: youjiali1995 <zlwgx1023@gmail.com>
/run-all-tests |
@youjiali1995 merge failed. |
/merge |
/run-all-tests |
@youjiali1995 merge failed. |
/run-integration-copr-test |
/merge |
/run-all-tests |
@youjiali1995 merge failed. |
/merge |
/run-all-tests |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #18550 |
Signed-off-by: youjiali1995 zlwgx1023@gmail.com
What problem does this PR solve?
Issue Number: close #18338
Problem Summary:
Green GC should scan locks on offline stores.
What is changed and how it works?
What's Changed:
Only skip tombstone stores.
Related changes
Check List
Tests
Release note