googleapis / googleapis/google-cloud-cpp
Refactor AsyncPollingLoop to accept an Operation directly
オープン
type: cleanup
- 主要言語
- C++
- スター
- 659
- フォーク
- 462
- 平均マージ
- 1日 2時間
- マージ済み PR(30日)
- 89
説明
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`.
コントリビューションガイド
調査の方向性
AsyncAwaitLongRunningOperation と AsyncPollingLoop から始め、future のキャンセルに関する既存の単体テストを確認します。現在、Operation がどのように future> にラップされているか、またキャンセルがどのように動作するかを追跡します。AsyncPollingLoop がラッパーなしでどちらの形式も受け入れ、キャンセルを含む影響を受ける単体テストがパスすれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- backend-api-design
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100