tursodatabase/libsql

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

Open

#1512 aperta il 27 giu 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C (490 fork)batch import
buggood first issuehelp wantedremote protocol

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Rivedere l'implementazione corrente di total changes() nel protocollo lato client Hrana. Esaminare come il flusso accumula le modifiche e comprendere perché il parametro di connessione non viene utilizzato a causa di un artificio generico. Cercare commenti esistenti o PR collegati per il contesto, quindi proporre una correzione per spostare l'accumulo a livello di connessione.
Tech stack
crust
Dominio
databasebackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70