nspcc-dev/neofs-node

`neofs-adm` waits for potentially useless transactions

Ouverte

#1 788 ouverte le 15 sept. 2022

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Go (49 forks)auto 404
I4S3U4discussiongood first issueneofs-adm

Métriques du dépôt

Stars
 (36 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Examinez la logique d'attente dans `cmd/neofs adm/internal/modules/morph/initialize.go` lignes 334-380. Comprenez le traitement des transactions et suggérez une modification pour s'arrêter sur toute erreur non HALT. Envisagez de simplifier la gestion des erreurs.
Stack technique
go
Domaine
backendblockchain
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45