googleapis / googleapis/google-cloud-cpp
Refactor AsyncPollingLoop to accept an Operation directly
- 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