FuelLabs/fuel-core

Separate Execution Gas Price and DA Gas Price into separate newtypes to add clarity

Open

Aperta il 22 ott 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)Rust (57.084 star) (2861 fork)batch import
good first issue

Descrizione

It's been suggested that we separate the two just to help organize the code a bit better. Instead of just dealing withu64 everywhere, it could help the reader to have types to communicate everywhere in the code the purpose of the numbers.

Guida contributor

Tech stack
rust
Dominio
backendblockchain
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
half day
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
blocked
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Basic Rust knowledgeUnderstanding of gas pricing in FuelFamiliarity with newtype pattern
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40
Direzione di ricerca
Rivedi la discussione nella PR #2364, in particolare il commento che suggeriva di separare i tipi. Quindi identifica tutte le occorrenze di `u64` che rappresentano i prezzi del gas nella codebase, distinguendo tra prezzo del gas di esecuzione e prezzo del gas DA. Crea due nuovi tipi, ad esempio `ExecutionGasPrice` e `DaGasPrice`, implementando i tratti necessari (es. aritmetici, serializzazione), e rifattorizza tutti gli usi per utilizzarli. Consulta la PR collegata per ulteriore contesto e potenziali insidie.