Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Backup: register backupTS as safepoint to PD #257

Merged
merged 2 commits into from
May 6, 2020

Conversation

3pointer
Copy link
Collaborator

What problem does this PR solve?

This PR can register backupTS as safePoint to PD, so the SafePoint won't move forward during backup. Avoid manually set GC safePoint.

What is changed and how it works?

Use pdClient.UpdateServiceSafePoint interface to lock down safePoint with TTL(60 secs)

Check List

Tests

  • No code

Related changes

  • Need to cherry-pick to the release branch
    need cherry-pick to release-4.0
  • Need to update the documentation
  • Need to be included in the release note

@3pointer 3pointer added the type/feature-request New feature or request label Apr 30, 2020
@3pointer 3pointer added this to the 4.0.0-GA milestone Apr 30, 2020
@codecov
Copy link

codecov bot commented Apr 30, 2020

Codecov Report

Merging #257 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #257   +/-   ##
=======================================
  Coverage   70.37%   70.37%           
=======================================
  Files          48       48           
  Lines        4949     4949           
=======================================
  Hits         3483     3483           
  Misses       1003     1003           
  Partials      463      463           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5b9632...123a76f. Read the comment docs.

tests/br_z_gc_safepoint/run.sh Outdated Show resolved Hide resolved
pkg/task/backup.go Outdated Show resolved Hide resolved
@3pointer 3pointer added WIP and removed WIP labels Apr 30, 2020
@3pointer 3pointer requested a review from kennytm May 6, 2020 10:09
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.

rest LGTM

pkg/task/backup.go Outdated Show resolved Hide resolved
tests/br_z_gc_safepoint/gc.go Outdated Show resolved Hide resolved
pkg/backup/client.go Outdated Show resolved Hide resolved
Copy link
Member

@overvenus overvenus left a comment

Choose a reason for hiding this comment

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

Rest LGTM

tests/br_z_gc_safepoint/run.sh Outdated Show resolved Hide resolved
@3pointer 3pointer merged commit ef93faa into pingcap:master May 6, 2020
3pointer added a commit to 3pointer/br that referenced this pull request May 6, 2020
Co-authored-by: kennytm <kennytm@gmail.com>
kennytm added a commit that referenced this pull request May 6, 2020
Co-authored-by: kennytm <kennytm@gmail.com>

Co-authored-by: kennytm <kennytm@gmail.com>
glkappe added a commit to glkappe/docs-cn that referenced this pull request Nov 12, 2020
Br adaptive GC description
pingcap/br#257
TomShawn added a commit to pingcap/docs-cn that referenced this pull request Nov 19, 2020
* Br adaptive GC description

Br adaptive GC description
pingcap/br#257

* Update backup-and-restore-use-cases.md

增加了修复版本链接, 请帮忙检查

* update

增加了修复版本链接, 请帮忙检查

* refine language

* Update br/backup-and-restore-use-cases.md

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/feature-request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants