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

Change the WAL journal mode back to the default DELETE mode #8037

Merged
merged 1 commit into from
May 24, 2024

Conversation

kozlovsky
Copy link
Collaborator

This PR fixes #8035.

The WAL mode is persistent, so it is necessary to switch it to DELETE mode at least once (see https://www.sqlite.org/wal.html).

After that, it is not strictly necessary to execute PRAGMA journal_mode = DELETE again, but executing it is cheap and fast, and consistently executing it every time the connection is open avoids some potential future errors.

@kozlovsky kozlovsky requested a review from a team as a code owner May 23, 2024 12:23
@kozlovsky kozlovsky requested review from drew2a and removed request for a team May 23, 2024 12:23
@kozlovsky kozlovsky force-pushed the fix/use_default_journal_mode branch from 722f2f1 to 5c2800a Compare May 24, 2024 09:40
@kozlovsky kozlovsky changed the title Change the WAL journal mode for the database beck to the default DELETE mode Change the WAL journal mode back to the default DELETE mode May 24, 2024
@kozlovsky kozlovsky merged commit 37b825f into Tribler:main May 24, 2024
15 of 19 checks passed
@kozlovsky kozlovsky deleted the fix/use_default_journal_mode branch May 24, 2024 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[7.14] Ten-second event loop freezes on sqlite3.Connection.disconnect call
2 participants