elastic / elastic/ml-cpp

[ML] Migrate CPersistUtils to use std::string_view

Aperta
#2,340 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
>enhancement
Lingua principale
C++
Stelle
157
Fork
67
Merge medio
17h 52m
PR unite (30g)
20

Descrizione

Now that main is on C++17 we can migrate `CPersistUtils` to use `std::string_view`. This would allow us to eliminate all string copies where we extract parts of a `std::string`.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia individuando la dichiarazione di CPersistUtils e tutti i punti di chiamata di estrazione delle stringhe. Traccia il modo in cui vengono utilizzate quelle parti di std::string, quindi migra le interfacce pertinenti a std::string_view mantenendo invariato il comportamento. Compila il progetto ed esegui la suite di test esistente per confermare che le copie di stringhe siano state rimosse senza regressioni.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
performance
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.