googleapis / googleapis/google-cloud-cpp

BigQuery V2 Minimal API Cleanup : Remove is_null from the ColumnData struct

Abierto
#14,387 0 comentarios 0 reacciones 0 asignados Ver en GitHub
type: cleanup
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

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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.