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

cache: Switch to apsw for cache SQLite db #58

Merged
merged 1 commit into from
May 23, 2024
Merged

Conversation

R2ZER0
Copy link
Contributor

@R2ZER0 R2ZER0 commented May 22, 2024

The bulitin sqlite3 library uses the version of sqlite3 linked to python, which is too old on the production datastore. This commit instead uses the apsw library which bundles a newer version of sqlite3.

The bulitin sqlite3 library uses the version of sqlite3 linked to
python, which is too old on the production datastore. This commit
instead uses the apsw library which bundles a newer version of sqlite3.
@R2ZER0 R2ZER0 requested a review from michaelwood May 22, 2024 17:28
Copy link
Member

@michaelwood michaelwood left a comment

Choose a reason for hiding this comment

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

LGTM fixes issue #51 ?

@R2ZER0 R2ZER0 merged commit ca95e7b into main May 23, 2024
6 checks passed
@R2ZER0 R2ZER0 deleted the rg/fix-cache-sqlite branch May 23, 2024 14:59
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.

2 participants