googleapis / googleapis/google-cloud-cpp
Generalize `RowKeyType::clear()`
- 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
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