tursodatabase/libsql

Total changes is accumulated per Hrana stream, not connection which is wrong

Open

#1 512 ouverte le 27 juin 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C (490 forks)batch import
buggood first issuehelp wantedremote protocol

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

We currently implement total_changes() incorrectly for Hrana client-side protocol because we accumulate the changes per stream, but should do it per connection. I merged the incorrect solution as an intermediate step to expose the API to users because fixing this is a bit involved. Hrana does pass connection to stream but turns it into something useless because of the generics trickery there.

Guide contributeur

Direction de recherche
Examiner l'implémentation actuelle de total changes() dans le protocole côté client Hrana. Étudier comment le flux accumule les changements et comprendre pourquoi le paramètre de connexion n'est pas utilisé en raison d'astuces de génériques. Chercher des commentaires existants ou des PR liés pour le contexte, puis proposer une correction pour déplacer l'accumulation au niveau de la connexion.
Stack technique
crust
Domaine
databasebackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70