algorand / algorand/conduit

Make `pipeline.NextRound` atomic

オープン
#140 コメント 0 件 リアクション 1 件 担当者 0 名 GitHub で見る
Team Lamprey
主要言語
Go
スター
44
フォーク
34
PR マージ指標
30日以内にマージされた PR はありません

説明

## Problem

The pipeline's **NextRound** is updated at the end of every round but accessed in several contexts that could be running in go-routines. A workaround we use ` atomic.LoadUint64` but that is no longer considered a best practice.

## Solution

Consider using [pipeline.NextRound.Load()](https://pkg.go.dev/sync/atomic#Uint64.Load) and similar functions for updating the variable.

### Caveat Emptor
This exact approach may not be backwards compatible on some 32-bit systems: https://go101.org/article/memory-layout.html

## Dependencies

Should wait until #128 is merged

## Urgency

Low

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。