Skip to content
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

Rc2.19.3 binary backup restore #550

Closed
wants to merge 22 commits into from
Closed

Rc2.19.3 binary backup restore #550

wants to merge 22 commits into from

Conversation

@oakeyc
Copy link
Contributor

@oakeyc oakeyc commented Feb 10, 2020

Move from: #548 so we could base the changes off of stable. Based on information from @piki

Original comment:
In this PR, we are trying to solve a few issues when backup and restore with binary backups

  1. Use pigz instead of gzip in backup-util
    Backup-util uses gzip which is super slow to compress large database files. It becomes the performance bottleneck when we backup binary backup. We will use pigz instead which is multi-threaded and way faster. (3 to 4 times faster for backing up large database)

  2. Backup-util tool should always pointing to sql master for binary backup restore
    This is an issue in current binary backup restore. Backup-util restores mysqldump from any instance in cluster because it connects to sql master through connection. Binary backup restore has different mechanism. We need to connect to the Sql master node and copy files there instead. Customer does not necessarily restore on the MySql Master node. (@lildude mentioned that we tell customers to restore from the MySQL master node, but seems the old logical backup restore works with non-master node so make this change to make sure it maintains same behavior)

/cc @github/ghes-infrastructure

@oakeyc oakeyc requested review from lildude, piki and jianghao0718 Feb 10, 2020
@jianghao0718
Copy link
Contributor

@jianghao0718 jianghao0718 commented Feb 10, 2020

I think you need to bump version to 2..19.3 for backup-utils/share/github-backup-utils/version. @lildude / @piki to confirm.

oakeyc and others added 3 commits Feb 10, 2020
Oakeyc/update binary
@jianghao0718 jianghao0718 reopened this Feb 11, 2020
@piki
Copy link
Contributor

@piki piki commented Feb 11, 2020

Please don't merge this PR. #551 was the right way to get these commits reviewed for a release. #548 is the right vehicle to get it into master. When you run script/release, it will create and immediately merge a PR to do the same thing this PR wants to do -- merge the commits into stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.