googleapis / googleapis/google-cloud-cpp

Generalize `RowKeyType::clear()`

Abierto
#15,248 0 comentarios 0 reacciones 0 asignados Ver en GitHub
api: bigtable type: cleanup
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.