tursodatabase/libsql

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

Open

#1,512 opened on 2024年6月27日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)C (490 forks)batch import
buggood first issuehelp wantedremote protocol

Repository metrics

Stars
 (16,744 stars)
PR merge metrics
 (平均マージ 23h 19m) (30d で 1 merged PR)

説明

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.

コントリビューターガイド

調査方針
Hrana クライアントサイドプロトコルにおける total changes() の現在の実装をレビューします。ストリームがどのように変更を累積するかを調べ、ジェネリクスのトリックにより接続パラメータが使用されない理由を理解します。既存のコメントや関連する PR を探して文脈を把握し、累積を接続レベルに移動する修正案を提案します。
技術スタック
crust
領域
databasebackend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GitC
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70