googleapis / googleapis/google-cloud-cpp
BigQuery V2 Minimal API Cleanup : Remove is_null from the ColumnData struct
- 主要言語
- C++
- スター
- 659
- フォーク
- 462
- 平均マージ
- 1日 2時間
- マージ済み PR(30日)
- 89
説明
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.
コントリビューションガイド
調査の方向性
google/cloud/bigquery/v2/minimal/internal/common_v2_resources.h の ColumnData から始め、意図された整理についてはリンクされている PR-14384 のレビュー フィードバックを読んでください。既存の is_null の使用箇所を追跡し、null 状態の表現を absl::optional に置き換えてください。別のパラメータなしで空文字列と null 値を区別できれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- databases
- issue の種類
- リファクタリング
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 50/100