`startUpdate` JavaDoc says "asynchronously" when it may not be
还没有人认领这个 Issue。
评估
- 难度
- 1/5
- 预计耗时
- 1 小时以内
- 新手友好度
- 45/100
- Issue 类型
- 文档
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- java
调研方向
打开 temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java 的第 704 行附近,检查 startUpdate 方法的 JavaDoc。查看其中如何描述 waitForStage 选项,然后更新文档,使 COMPLETED 明确表示调用可能会阻塞;当 JavaDoc 不再暗示该方法始终是异步的时,即表示完成。
由索引模型根据 Issue 内容生成。
描述
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!
- 主要语言
- Java
- 星标
- 433
- 派生
- 249
- 平均合并
- 6 天 5 小时
- 30 天内合并 PR
- 25
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
temporalio/sdk-java 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 62/100
temporalio/sdk-java#2676 · 8 条评论 · 2 个 reaction ·
-
enhancement
难度 2/5 1-3 小时 新手友好度 65/100
temporalio/sdk-java#1825 ·
-
test server
难度 4/5 3-5 天 新手友好度 42/100
temporalio/sdk-java#3088 · 2 条评论 ·
-
temporalio/sdk-java#3059 · 已指派 1 人 ·
-
enhancement
temporalio/sdk-java#3058 · 已指派 1 人 ·
查看 temporalio/sdk-java 的全部 Issue
相似的 Issue
-
area-deployment area-integrations triage:bot-seen
难度 2/5 半天 新手友好度 86/100
-
难度 2/5 1-3 小时 新手友好度 75/100
apache/flink-agents#1156 ·
-
area/connectors autoteam community connectors/source/shopify needs-triage team/use type/bug
难度 2/5 1-3 小时 新手友好度 84/100
-
难度 2/5 1-3 小时 新手友好度 70/100
-
难度 1/5 1 小时以内 新手友好度 85/100