googleapis / googleapis/google-cloud-cpp

Refactor AsyncPollingLoop to accept an Operation directly

Abierto
#14,336 0 comentarios 0 reacciones 0 asignados Ver en GitHub
type: cleanup
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.