googleapis / googleapis/google-cloud-cpp

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

Ouverte
#14,387 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
type: cleanup
Langage dominant
C++
Étoiles
659
Forks
462
Merge moyen
1 j 2 h
PR mergées (30 j)
89

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par ColumnData dans google/cloud/bigquery/v2/minimal/internal/common_v2_resources.h et lisez les commentaires de revue liés de PR-14384 pour connaître le nettoyage prévu. Suivez les utilisations existantes de is_null et remplacez la représentation de l’état nul par absl::optional ; le travail est considéré comme terminé lorsque les chaînes vides restent distinguables des valeurs nulles sans le paramètre séparé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
databases
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
50/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.