tursodatabase/libsql

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

Open

#1.512 aberto em 27 de jun. de 2024

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C (490 forks)batch import
buggood first issuehelp wantedremote protocol

Métricas do repositório

Stars
 (16.744 stars)
Métricas de merge de PR
 (Mesclagem média 23h 19m) (1 fundiu PR em 30d)

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.

Guia do colaborador

Direção de pesquisa
Revisar a implementação atual de total changes() no protocolo do lado do cliente Hrana. Examinar como o fluxo acumula alterações e entender por que o parâmetro de conexão não é usado devido a truques de genéricos. Procurar comentários existentes ou PRs vinculados para contexto e, em seguida, propor uma correção para mover o acúmulo para o nível de conexão.
Pilha de tecnologia
crust
Domain
databasebackend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70