`startUpdate` JavaDoc says "asynchronously" when it may not be
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- java
- Área
- documentation
Línea de trabajo
Abre temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java alrededor de la línea 704 e inspecciona el JavaDoc de los métodos startUpdate. Comprueba cómo se describe la opción waitForStage y, después, actualiza la documentación para que COMPLETED indique claramente que la llamada puede bloquearse; el trabajo estará terminado cuando el JavaDoc ya no implique que el método es siempre asíncrono.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Expected Behavior
The JavaDoc for startUpdate should indicate that when it's called with a waitForStage of WorkflowUpdateStage.COMPLETED, it's synchronous and blocks until the update is completed.
Actual Behavior
The JavaDoc for startUpdate says the following:
Start a zero argument update workflow request asynchronously.
However, the following code will block until the update is completed:
val updateOptions =
UpdateOptions.newBuilder<UpdateResult>()
.setResultClass(UpdateResult::class.java)
.setWaitForStage(WorkflowUpdateStage.COMPLETED)
.setUpdateId(updateId)
.build()
val update = WorkflowClient.startUpdate(stub::doUpdate, updateOptions) // <-- blocks here
val result = update.getResult(5, TimeUnit.SECONDS)
The JavaDoc implies that startUpdate would ignore the waitForStage, always be asynchronous, and block on the update.getResult.
Steps to Reproduce the Problem
- Look at JavaDoc for
startUpdate - Assume
startUpdateis always asynchronous - Call
startUpdatewith awaitForStageofWorkflowUpdateStage.COMPLETED - See that the
startUpdatecall was actually synchronous
Specifications
- Version: latest master
- Platform: n/a
Requesting that either the word asynchronously is removed from the JavaDoc for the startUpdate methods or it's clear that the function isn't asynchronous in a normal sense.
Thanks!
- Lenguaje dominante
- Java
- Estrellas
- 433
- Forks
- 249
- Merge medio
- 6 d 5 h
- PR fusionados (30 d)
- 25
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de temporalio/sdk-java
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
temporalio/sdk-java#2676 · 8 comentarios · 2 reacciones ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
temporalio/sdk-java#1825 ·
-
test server
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
temporalio/sdk-java#3088 · 2 comentarios ·
-
temporalio/sdk-java#3059 · 1 asignado ·
-
enhancement
temporalio/sdk-java#3058 · 1 asignado ·
Todos los issues de temporalio/sdk-java
Issues similares
-
area-deployment area-integrations triage:bot-seen
Dificultad 2/5 Medio día Aptitud para principiantes 86/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
apache/flink-agents#1156 ·
-
[source-shopify] FAILED bulk operation without partialDataUrl is silently treated as successful Abiertoarea/connectors autoteam community connectors/source/shopify needs-triage team/use type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100