Skip to content
This repository has been archived by the owner on Dec 8, 2021. It is now read-only.

restore: update and restore GCLifeTime once when parallel (#220) #224

Merged
merged 3 commits into from
Aug 7, 2019

Conversation

lance6716
Copy link
Contributor

What problem does this PR solve?

Cherry-pick #220 to release-2.1.

What is changed and how it works?

Check List

Tests

  • Unit test
  • Integration test

Side effects

Related changes

* restore: update and restore GCLifeTime once when parallel

* Update lightning/restore/restore.go

Co-Authored-By: amyangfei <amyangfei@gmail.com>

* Update lightning/restore/restore.go

Co-Authored-By: amyangfei <amyangfei@gmail.com>

* Update lightning/restore/restore.go

Co-Authored-By: amyangfei <amyangfei@gmail.com>

* restore: fix bug in reviewing

* restore: call ObtainGCLifeTime closer to use

* restore: improve readabiilty

* restore: reduce struct copy and pointer deref

* restore: fix bug in reviewing

* Update lightning/restore/restore.go

Co-Authored-By: kennytm <kennytm@gmail.com>

* restore: improve readability

* restore: refine mock expectation order

* restore: adjust detail
* restore: update and restore GCLifeTime once when parallel

* Update lightning/restore/restore.go

Co-Authored-By: amyangfei <amyangfei@gmail.com>

* Update lightning/restore/restore.go

Co-Authored-By: amyangfei <amyangfei@gmail.com>

* Update lightning/restore/restore.go

Co-Authored-By: amyangfei <amyangfei@gmail.com>

* restore: fix bug in reviewing

* restore: call ObtainGCLifeTime closer to use

* restore: improve readabiilty

* restore: reduce struct copy and pointer deref

* restore: fix bug in reviewing

* Update lightning/restore/restore.go

Co-Authored-By: kennytm <kennytm@gmail.com>

* restore: improve readability

* restore: refine mock expectation order

* restore: adjust detail
@lance6716 lance6716 added 2.1-cherry-pick The target branch is release-2.1 priority/normal status/PTAL This PR is ready for review. Add this label back after committing new changes type/cherry-pick Cherry-pick (backport) labels Aug 7, 2019
@lance6716
Copy link
Contributor Author

/run-all-tests tidb=release-2.1 pd=release-2.1 tikv=release-2.1

PTAL @kennytm @leoppro

Copy link
Collaborator

@kennytm kennytm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kennytm kennytm added the status/LGT1 One reviewer already commented LGTM (LGTM1) label Aug 7, 2019
@zier-one
Copy link
Contributor

zier-one commented Aug 7, 2019

LGTM

@zier-one zier-one added status/LGT2 Two reviewers already commented LGTM, ready for merge (LGTM2) and removed status/LGT1 One reviewer already commented LGTM (LGTM1) labels Aug 7, 2019
@kennytm kennytm merged commit f010cd8 into release-2.1 Aug 7, 2019
@kennytm kennytm deleted the lance6716/cherry-pick-220 branch August 7, 2019 09:02
@lance6716 lance6716 changed the title Lance6716/cherry pick 220 restore: update and restore GCLifeTime once when parallel (#220) Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.1-cherry-pick The target branch is release-2.1 status/LGT2 Two reviewers already commented LGTM, ready for merge (LGTM2) status/PTAL This PR is ready for review. Add this label back after committing new changes type/cherry-pick Cherry-pick (backport)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants