nspcc-dev/neofs-node

`neofs-adm` waits for potentially useless transactions

Aperta

#1788 aperta il 15 set 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Go (49 fork)auto 404
I4S3U4discussiongood first issueneofs-adm

Metriche repository

Star
 (36 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

It is common practice to send some TXs, cache their hashes, and wait for their chain inclusion.

Adm CLI stops if one of the TX has not been persisted but does not stop if one of the TX was failed. Moreover, we return only the first not-HALT-err and drop other errors.

I suggest just stopping iterating on any non-HALT TX. It also would simplify the code.

Guida contributor

Direzione di ricerca
Esamina la logica di attesa nel file `cmd/neofs adm/internal/modules/morph/initialize.go` alle righe 334-380. Comprendi la gestione delle transazioni e proponi una modifica per fermarsi su qualsiasi errore non HALT. Considera di semplificare la gestione degli errori.
Tech stack
go
Dominio
backendblockchain
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.
Abbastanza chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45