tursodatabase/libsql

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

Open

#1.512 geöffnet am 27. Juni 2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C (490 Forks)batch import
buggood first issuehelp wantedremote protocol

Repository-Metriken

Stars
 (16.744 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 23h 19m) (1 gemergte PR in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie die aktuelle Implementierung von total changes() im Hrana Clientseitenprotokoll. Untersuchen Sie, wie der Stream Änderungen akkumuliert, und verstehen Sie, warum der Verbindungsparameter aufgrund von Generics Tricks nicht verwendet wird. Suchen Sie nach vorhandenen Kommentaren oder verknüpften PRs für Kontext und schlagen Sie dann eine Korrektur vor, um die Akkumulation auf die Verbindungsebene zu verschieben.
Tech Stack
crust
Domain
databasebackend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70