googleapis / googleapis/google-cloud-cpp

Generalize `RowKeyType::clear()`

Offen
#15,248 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
api: bigtable type: cleanup
Vorherrschende Sprache
C++
Sterne
659
Forks
462
Ø Merge
1 T. 2 Std.
Gemergte PRs (30 T.)
89

Beschreibung

`bigtable::RowKeyType` is a `std::string` in OSS-land, and an `absl::Cord` in Google-land.

`absl::Cord::clear()` is deprecated, in favor of `absl::Cord::Clear()`. `std::string::Clear()` is not a thing.

We need to add a new `Clear(RowKeyType&)` function to this file:

https://github.com/googleapis/google-cloud-cpp/blob/main/google/cloud/bigtable/internal/google_bytes_traits.h

which calls `std::string::clear()` in OSS-land, and `absl::Cord::Clear()` in Google-land.

This unblocks a LSC in Google-land.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in google/cloud/bigtable/internal/google_bytes_traits.h und untersuche die vorhandene Behandlung von RowKeyType sowie die mit Clear verbundenen Hilfsfunktionen. Füge die angeforderte Clear(RowKeyType&) Abstraktion mit dem im Issue beschriebenen Verhalten für OSS und Google-Land hinzu und überprüfe anschließend, dass die relevanten Projektprüfungen in beiden Konfigurationen erfolgreich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
database
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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