temporalio / temporalio/features
Add unregistered_activity_invocation
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 32
- Forks
- 28
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 7
Description
Currently Go Sdk increments unregistered_activity_invocation counter metric when a worker picks up activity task for activity it does not have registered.
Having this metric is pretty useful to have and alert in prod envs where a small mistake of registering
activity can lead to activity failure after all retries.
Feature request is ask to add either new metric or add a new flag to activity_task_failed maybe
that shows indication that failure was due to unregistered activity on worker - https://github.com/temporalio/sdk-java/blob/master/temporal-sdk/src/main/java/io/temporal/internal/activity/ActivityTaskHandlerImpl.java#L111
Contributor guide
No contributing guide indexed for this repository
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 with the linked Java ActivityTaskHandlerImpl implementation, then locate the corresponding activity-task handling path in the Go SDK. Determine whether the behavior should be a new unregistered-activity metric or an indication on activity_task_failed; done means the unregistered case is distinguishable for production alerting and covered by the relevant tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- backend, observability
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100