googleapis / googleapis/google-cloud-cpp

Refactor AsyncPollingLoop to accept an Operation directly

オープン
#14,336 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
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

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

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