googleapis / googleapis/google-cloud-cpp
BigQuery V2 Minimal API Cleanup : Remove is_null from the ColumnData struct
- 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