Skip to content

Use a readonly connection to create the dump#121

Merged
guvra merged 1 commit intomainfrom
feat-readonly-connection
Mar 5, 2024
Merged

Use a readonly connection to create the dump#121
guvra merged 1 commit intomainfrom
feat-readonly-connection

Conversation

@guvra
Copy link
Copy Markdown
Collaborator

@guvra guvra commented Mar 5, 2024

This PR adds the following query to the init_commands array passed to mysqldump-php:
SET SESSION TRANSACTION READ ONLY

@guvra guvra requested a review from rbayet March 5, 2024 10:57
@guvra guvra self-assigned this Mar 5, 2024
@guvra guvra added the feature New feature or request label Mar 5, 2024
@guvra guvra requested a review from PierreGauthier March 5, 2024 11:01
@guvra guvra force-pushed the feat-readonly-connection branch from 15121c6 to 7e290b9 Compare March 5, 2024 14:15
Copy link
Copy Markdown
Contributor

@staabm staabm left a comment

Choose a reason for hiding this comment

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

nice

@guvra guvra merged commit aa034ae into main Mar 5, 2024
@guvra guvra deleted the feat-readonly-connection branch March 5, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants