nspcc-dev/neofs-node

`neofs-adm` waits for potentially useless transactions

Offen

#1.788 geöffnet am 15.09.2022

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (49 Forks)auto 404
I4S3U4discussiongood first issueneofs-adm

Repository-Metriken

Stars
 (36 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die Wartelogik in `cmd/neofs adm/internal/modules/morph/initialize.go` Zeilen 334-380. Verstehe die Transaktionsverarbeitung und schlage eine Änderung vor, um bei jedem Nicht HALT Fehler anzuhalten. Erwäge die Vereinfachung der Fehlerbehandlung.
Tech Stack
go
Domain
backendblockchain
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45