elastic / elastic/ml-cpp

[ML] Migrate CPersistUtils to use std::string_view

Offen
#2,340 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
>enhancement
Vorherrschende Sprache
C++
Sterne
157
Forks
67
Ø Merge
17 Std. 52 Min.
Gemergte PRs (30 T.)
20

Beschreibung

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`.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, die Deklaration von CPersistUtils und alle Aufrufstellen der String-Extraktion zu finden. Verfolge, wie diese Teile von std::string verwendet werden, und migriere dann die relevanten Schnittstellen zu std::string_view, wobei das Verhalten erhalten bleibt. Baue das Projekt und führe die vorhandene Testsuite aus, um zu bestätigen, dass String-Kopien ohne Regressionen entfernt wurden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
performance
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.