tursodatabase/libsql

Use `WAL` as default everywhere

Open

#1,553 opened on Jul 14, 2024

View on GitHub
 (5 comments) (0 reactions) (0 assignees)C (16,744 stars) (490 forks)batch import
good first issue

Description

Currently, Turso (the platform), libsql-sever, and embedded replicas use WAL mode and it is the only mode supported. However libsql drivers don't have WAL mode as default. Let's change that to use WAL everywhere.

reported here: https://discord.com/channels/933071162680958986/1154725980183732224/1261642078694150144

Contributor guide

Tech stack
csql
Domain
databasebackend
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
CSQLite internalslibsql driver architecture
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
The issue asks to set WAL mode as the default for libsql drivers. To implement, first identify the driver source files (likely in C or Rust) where the connection or database is initialized. Look for existing pragma settings like 'journal mode' and change to 'WAL'. Check if there are any compatibility concerns across platforms or embedded replicas. The Discord link may provide context, but the core change is straightforward: modify the default pragma in driver initialization code.