tursodatabase/libsql

Improve database connection builder API?

Open

#1635 aperta il 5 ago 2024

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C (490 fork)batch import
Embedded Replicasenhancementhelp wantedquestion

Metriche repository

Star
 (16.744 star)
Metriche merge PR
 (Merge medio 23h 19m) (1 PR mergiata in 30 g)

Descrizione

The new_remote_replica() and new_local_replica() names are weird because in both cases the replica is local. The difference is just with how they are synced.

My proposal is to keep new_local() and new_remote() but add additional with_remote_sync() and with_file_sync() methods to the builder for local database variant.

Guida contributor

Direzione di ricerca
Esplora l'SDK client Rust per libsql, in particolare l'implementazione del costruttore di connessioni al database. Esamina le funzioni esistenti `new remote replica()` e `new local replica()` nel codice sorgente (probabilmente in un modulo builder). Comprendi il pattern builder utilizzato e come aggiungere metodi come `with remote sync()` e `with file sync()`. Fai riferimento ai commenti dell'issue per eventuali feedback dei maintainer.
Tech stack
rustsql
Dominio
database
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
95