googleapis / googleapis/google-cloud-cpp
BigQuery V2 Minimal API Cleanup : Remove is_null from the ColumnData struct
- Ngôn ngữ chính
- C++
- Star
- 659
- Fork
- 462
- Merge trung bình
- 1 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 89
Mô tả
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.
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu với ColumnData trong google/cloud/bigquery/v2/minimal/internal/common_v2_resources.h và đọc phản hồi review được liên kết của PR-14384 để biết phần dọn dẹp dự kiến. Truy vết các cách sử dụng is_null hiện có và thay thế biểu diễn trạng thái null bằng absl::optional; hoàn thành khi các chuỗi rỗng vẫn có thể phân biệt với các giá trị null mà không cần tham số riêng biệt.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- databases
- Loại issue
- Tái cấu trúc
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 50/100