temporalio / temporalio/sdk-java
Provide WorkflowStatus in WorkflowNotFoundException when signing a workflow
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 433
- Forks
- 249
- Avg merge
- 5d 6h
- Merged PRs (30d)
- 26
Description
Is your feature request related to a problem? Please describe.
Currently signaling a closed workflow will throw this exception, but there are several cases:
- Workflow really doesn't exist (or deleted after retention)
- Workflow is completed/terminated/etc (closed)
As an application, we sometimes need to handle the two cases differently(error handling).
Describe the solution you'd like
The exception should just return "WorkflowStatus" inside.
Describe alternatives you've considered
Currently we have to make a DescribeWorkflowExecution API call to achieve this. This requires more code, more error handling, and higher latency .
Additional context
Add any other context or screenshots about the feature request here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the Java SDK's WorkflowNotFoundException handling for signing a workflow and identify where the workflow status is available. Add coverage for the distinction between a missing workflow and a closed workflow, with done meaning the exception exposes WorkflowStatus without requiring a separate DescribeWorkflowExecution call.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- api, backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100