googleapis / googleapis/google-cloud-cpp

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

オープン
#14,387 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
type: cleanup
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。