googleapis / googleapis/google-cloud-cpp
BigQuery V2 Minimal API Cleanup : Remove is_null from the ColumnData struct
- Lenguaje dominante
- C++
- Estrellas
- 659
- Forks
- 462
- Merge medio
- 1 d 2 h
- PR fusionados (30 d)
- 89
Descripción
This cleanup is result of code review request for PR-14384 . More details [here](https://github.com/googleapis/google-cloud-cpp/pull/14384#pullrequestreview-2146018253).
Basically, an `is_null` parameter was added to[ `ColumnData` struct](https://github.com/googleapis/google-cloud-cpp/blob/8d01ae34c018c34a1002187705385775e70a4113/google/cloud/bigquery/v2/minimal/internal/common_v2_resources.h#L317) to distinguish between empty strings and null values.
The cleanup is to use `absl::optional` instead as mentioned in the code review feedback linked above.
Guía de contribución
Línea de trabajo
Comienza con ColumnData en google/cloud/bigquery/v2/minimal/internal/common_v2_resources.h y lee los comentarios de revisión enlazados de PR-14384 para conocer la limpieza prevista. Rastrea los usos existentes de is_null y reemplaza la representación del estado nulo por absl::optional; se considera terminado cuando las cadenas vacías siguen siendo distinguibles de los valores nulos sin el parámetro separado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- databases
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 50/100