This repository was archived by the owner on Jul 24, 2024. It is now read-only.
Merged
Conversation
…gcap#124) Signed-off-by: Neil Shen <overvenus@gmail.com>
* restore: support batch split Signed-off-by: 5kbpers <tangminghua@pingcap.com> * go mod tidy Signed-off-by: 5kbpers <tangminghua@pingcap.com> * address lint Signed-off-by: 5kbpers <tangminghua@pingcap.com> Co-authored-by: Neil Shen <overvenus@gmail.com>
Remove deployment section since BR is a command-line tool, and there are no special deployment requirements.
* restore: remove/add pd scheduler in restore * add test * address comment * address comment Co-authored-by: Neil Shen <overvenus@gmail.com>
* conn: fix inaccurate region count Signed-off-by: Neil Shen <overvenus@gmail.com> * conn: add getRegionCount test Signed-off-by: Neil Shen <overvenus@gmail.com>
* validate: debug backupmeta * address comment
Signed-off-by: 5kbpers <tangminghua@pingcap.com>
…cap#109) * cmd: add lastbackupts arguments to full and table backup Signed-off-by: Yang Keao <keao.yang@yahoo.com> * cmd: add lastbackupts arguments to full and database and table restore Signed-off-by: Yang Keao <keao.yang@yahoo.com> * restore: set NewTimestamp for downloadSST rewrite rule Signed-off-by: Yang Keao <keao.yang@yahoo.com> * restore: reset PD's ts if lastBackupTS is zero Signed-off-by: Yang Keao <keao.yang@yahoo.com> * restore: restore from a backupmeta is good enough Signed-off-by: Yang Keao <keao.yang@yahoo.com> * cmd/validate: add parse cmd Signed-off-by: 5kbpers <tangminghua@pingcap.com> * restore/db: insert `IF NOT EXISTS` into create table SQL Signed-off-by: 5kbpers <tangminghua@pingcap.com> * tests: add integration test for incremental br Signed-off-by: 5kbpers <tangminghua@pingcap.com> * *: update kvproto and remove \t in error Signed-off-by: Yang Keao <keao.yang@yahoo.com> * fix bugs in test script Signed-off-by: Yang Keao <keao.yang@yahoo.com> * address comments Signed-off-by: 5kbpers <tangminghua@pingcap.com> Co-authored-by: 5kbpers <20279863+5kbpers@users.noreply.github.com>
Signed-off-by: Neil Shen <overvenus@gmail.com>
059f217 to
bbe089d
Compare
* flags: hidden not stable command flags * address comment
3pointer
reviewed
Jan 3, 2020
|
|
||
| echo "start version: $start_ts, end version: $end_ts" | ||
|
|
||
| # incremental restore |
Collaborator
There was a problem hiding this comment.
It seems we did not cleanup data after Line43, so it always success even though restore failed?
3pointer
approved these changes
Jan 3, 2020
* restore: support table partition Signed-off-by: 5kbpers <tangminghua@pingcap.com> * add test Signed-off-by: 5kbpers <tangminghua@pingcap.com> * fix get rewrite rules Signed-off-by: 5kbpers <tangminghua@pingcap.com> * fix checksum Signed-off-by: 5kbpers <tangminghua@pingcap.com> * add warn log Signed-off-by: 5kbpers <tangminghua@pingcap.com> * reset tidb-tools Signed-off-by: 5kbpers <tangminghua@pingcap.com> * add check for validate file Signed-off-by: 5kbpers <tangminghua@pingcap.com> * replace zap.ByteString with zap.Binary Signed-off-by: 5kbpers <tangminghua@pingcap.com>
ed4c5d1 to
b412109
Compare
Codecov Report
@@ Coverage Diff @@
## release-3.1 #130 +/- ##
==============================================
Coverage ? 72.95%
==============================================
Files ? 33
Lines ? 3257
Branches ? 0
==============================================
Hits ? 2376
Misses ? 591
Partials ? 290
Continue to review full report at Codecov.
|
3pointer
reviewed
Jan 7, 2020
Signed-off-by: Neil Shen <overvenus@gmail.com>
b412109 to
b4ba91e
Compare
overvenus
pushed a commit
to overvenus/br-1
that referenced
this pull request
Dec 29, 2020
* add badge * replace build to test
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge recent commits into release-3.1