indeedeng / indeedeng/iwf-java-sdk

[Maybe] Unify error handling for waitForWorkflowCompletion and tryGettingWorkflowResults

Abierto
#202 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Java
Estrellas
17
Forks
8
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Today the withWait api will throw client side exception with long poll timeout sub status . Because this is a 4xx error from server.

https://github.com/indeedeng/iwf-java-sdk/blob/3409ea77f0b0764f0dbe79670ecd48a0aff9609e/src/test/java/io/iworkflow/integ/WorkflowUncompletedTest.java#L44

However, tryGetting api will throw UncompletedException.

this inconsistency could be a bit confusing.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con el test referenciado integ/WorkflowUncompletedTest.java alrededor de la línea 44 y compara los puntos de entrada de workflow-result withWait y tryGetting. Comprueba cómo informa cada uno de un workflow incompleto y, después, aclara con los maintainers cuál es el comportamiento común previsto; el trabajo estará terminado cuando la inconsistencia se haya resuelto y esté cubierta por tests.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
backend-api-design
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
30/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.