temporalio / temporalio/sdk-java
WorkerOptions should enforce an amount of executors to be not less than an amount of pollers
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 433
- Forks
- 249
- Avg merge
- 5d 6h
- Merged PRs (30d)
- 26
Description
Expected Behavior
WorkerOptions should enforce an amount of executors to be not less than an amount of pollers.
Actual Behavior
Currently, users can create WorkerOptions with maxConcurrentWorkflowTaskExecutionSize = 1 which is effectively an invalid configuration with the worker not being able to open at least one sticky and 1 normal queue poll. Every worker needs at least 2 executors: one to be reserved and allow polling on the main task queue and another to be reserved and allow polling on a sticky task queue.
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 at WorkerOptions and locate its existing configuration validation and related tests. Done means a configuration with fewer executors than pollers, including maxConcurrentWorkflowTaskExecutionSize = 1, is rejected.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100