googleapis / googleapis/google-cloud-cpp
Generalize `RowKeyType::clear()`
- Lenguaje dominante
- C++
- Estrellas
- 659
- Forks
- 462
- Merge medio
- 1 d 2 h
- PR fusionados (30 d)
- 89
Descripción
`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.
Guía de contribución
Línea de trabajo
Comienza en google/cloud/bigtable/internal/google_bytes_traits.h e inspecciona el manejo existente de RowKeyType y los helpers relacionados con Clear. Añade la abstracción Clear(RowKeyType&) solicitada con el comportamiento para OSS y Google-land descrito en el issue, y verifica después que las comprobaciones relevantes del proyecto pasen en ambas configuraciones.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- database
- Tipo de issue
- Refactorización
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 45/100