googleapis / googleapis/google-cloud-cpp

Refactor AsyncPollingLoop to accept an Operation directly

Đang mở
#14,336 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
type: cleanup
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ả

Currently in `AsyncAwaitLongRunningOperation` we can construct a `future>` from an `Operation` just so we can pass it to `AsyncPollingLoop`. It would be preferable to refactor `AsyncPollingLoop` such that it could accept either.

A brief attempt to do this refactoring was made, but resulted in unit test failures when trying to cancel the `future`.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu với AsyncAwaitLongRunningOperation và AsyncPollingLoop, sau đó kiểm tra các unit test hiện có liên quan đến việc hủy future. Theo dõi cách một Operation hiện được bọc trong future> và việc hủy hoạt động như thế nào. Hoàn thành khi AsyncPollingLoop chấp nhận cả hai dạng mà không có wrapper và các unit test bị ảnh hưởng đều đạt, bao gồm cả việc hủy.

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
backend-api-design
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
42/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.