tursodatabase/libsql

Improve database connection builder API?

Open

#1 635 ouverte le 5 août 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C (490 forks)batch import
Embedded Replicasenhancementhelp wantedquestion

Métriques du dépôt

Stars
 (16 744 stars)
Métriques de merge PR
 (Merge moyen 23h 19m) (1 PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Explorez le SDK client Rust pour libsql, en particulier l'implémentation du constructeur de connexion à la base de données. Examinez les fonctions existantes `new remote replica()` et `new local replica()` dans le code source (probablement dans un module builder). Comprenez le pattern builder utilisé et comment ajouter des méthodes comme `with remote sync()` et `with file sync()`. Référez vous aux commentaires de l'issue pour tout retour des mainteneurs.
Stack technique
rustsql
Domaine
database
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
95