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

BackupDb streams data from database #894

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Quetzacoalt91
Copy link
Member

Questions Answers
Description? Following an ugprade attempt made on a production like shop, we discovered the database backup takes too much time. It did not complete after a whole day of work while a save of the same database via PhpMyAdmin takes a couple of minutes. This PRs is a second attempt to see an improvement on the time needed to backup de database.
Type? improvement
BC breaks? Nope
Deprecations? Nope
Fixed ticket? /
Sponsor company @PrestaShopCorp
How to test? Backups are faster

@Quetzacoalt91 Quetzacoalt91 self-assigned this Sep 13, 2024
@Quetzacoalt91 Quetzacoalt91 force-pushed the stream-backupdb-data branch 2 times, most recently from 2ae8a24 to 4fb33cd Compare September 16, 2024 15:36
@Quetzacoalt91 Quetzacoalt91 marked this pull request as ready for review September 17, 2024 12:55
@Quetzacoalt91 Quetzacoalt91 force-pushed the stream-backupdb-data branch 3 times, most recently from e1d0caa to d00704a Compare September 17, 2024 15:39
@Quetzacoalt91
Copy link
Member Author

Restoring the files generated by this PR crashes the database with ERROR 2006 (HY000) at line 27: Server has gone away. I need to split the values in several INSERT INTO statements.

Copy link

sonarcloud bot commented Sep 19, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Ready for review
Development

Successfully merging this pull request may close these issues.

1 participant