FuelLabs/fuel-core

Avoid cloning of vectors in `HexString`

Open

Aperta il 1 feb 2025

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

Descrizione

The GraphQL server coverts bytes into the hex string to return it to the user. During this convert process, we take vector by value.

We have many cases where we clone vectors instead of passing it as a reference.

This issue is created to replace Vec<u8> with Cow<[u8]> or Cow<Vec<u8>> and allow usage of references.

Guida contributor

Tech stack
rust
Dominio
backendapiperformance
Tipo issue
performance
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.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
mostly clear
Prerequisiti
Rust ownershipCow<[u8]> pattern
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
Il problema riguarda la sostituzione di `Vec<u8>` con `Cow<[u8]>` o `Cow<Vec<u8>>` nella conversione di stringhe esadecimali all'interno del server GraphQL. Inizia cercando le funzioni che prendono `Vec<u8>` per valore e chiamano `clone()` su di esso. Esamina la logica di conversione nel codice del server GraphQL, probabilmente in file relativi alla codifica esadecimale. Controlla i commenti e le PR esistenti per eventuali discussioni o approcci precedenti.