googleapis / googleapis/google-cloud-cpp
Refactor AsyncPollingLoop to accept an Operation directly
- Lenguaje dominante
- C++
- Estrellas
- 659
- Forks
- 462
- Merge medio
- 1 d 2 h
- PR fusionados (30 d)
- 89
Descripción
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`.
Guía de contribución
Línea de trabajo
Comienza con AsyncAwaitLongRunningOperation y AsyncPollingLoop; después, inspecciona las pruebas unitarias existentes sobre la cancelación de futures. Traza cómo se envuelve actualmente un Operation en future> y cómo se comporta la cancelación. Se considera terminado cuando AsyncPollingLoop acepta cualquiera de las dos formas sin el wrapper y pasan las pruebas unitarias afectadas, incluida la cancelación.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- backend-api-design
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100