googleapis / googleapis/google-cloud-cpp

Refactor AsyncPollingLoop to accept an Operation directly

Ouverte
#14,336 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
type: cleanup
Langage dominant
C++
Étoiles
659
Forks
462
Merge moyen
1 j 2 h
PR mergées (30 j)
89

Description

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`.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par AsyncAwaitLongRunningOperation et AsyncPollingLoop, puis examinez les tests unitaires existants concernant l’annulation des futures. Suivez la façon dont un Operation est actuellement encapsulé dans future> et le comportement de l’annulation. Le travail est terminé lorsque AsyncPollingLoop accepte l’une ou l’autre forme sans le wrapper et que les tests unitaires concernés passent, y compris ceux sur l’annulation.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
backend-api-design
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.