FuelLabs/fuel-core

Avoid cloning of vectors in `HexString`

Open

#2.657 geöffnet am 1. Feb. 2025

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (2.861 Forks)batch import
good first issue

Repository-Metriken

Stars
 (57.084 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8h 53m) (1 gemergte PR in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Problem betrifft das Ersetzen von `Vec<u8>` durch `Cow<[u8]>` oder `Cow<Vec<u8>>` bei der Hex String Konvertierung im GraphQL Server. Beginnen Sie mit der Suche nach Funktionen, die `Vec<u8>` als Wert entgegennehmen und `clone()` darauf aufrufen. Untersuchen Sie die Konvertierungslogik im GraphQL Server Code, wahrscheinlich in Dateien im Zusammenhang mit Hex Kodierung. Überprüfen Sie vorhandene Kommentare und PRs auf frühere Diskussionen oder Ansätze.
Tech Stack
rust
Domain
backendapiperformance
Issue Type
Performance
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.
Halber Tag
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
Rust ownershipCow<[u8]> pattern
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70