FuelLabs/fuel-core

Avoid cloning of vectors in `HexString`

Open

#2 657 ouverte le 1 févr. 2025

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Rust (2 861 forks)batch import
good first issue

Métriques du dépôt

Stars
 (57 084 stars)
Métriques de merge PR
 (Merge moyen 8h 53m) (1 PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Le problème implique de remplacer `Vec<u8>` par `Cow<[u8]>` ou `Cow<Vec<u8>>` dans la conversion de chaîne hexadécimale au sein du serveur GraphQL. Commencez par rechercher les fonctions qui prennent `Vec<u8>` par valeur et appellent `clone()` dessus. Examinez la logique de conversion dans le code du serveur GraphQL, probablement dans les fichiers liés à l'encodage hexadécimal. Vérifiez les commentaires et PR existants pour toute discussion ou approche antérieure.
Stack technique
rust
Domaine
backendapiperformance
Type d'issue
Performance
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.
Une demi journée
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
Rust ownershipCow<[u8]> pattern
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70