temporalio / temporalio/api

[Feature Request] Remove RetryState.RETRY_STATE_IN_PROGRESS from public api

Open
#232 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Go
Stars
133
Forks
101
Avg merge
1d 22h
Merged PRs (30d)
11

Description

RetryState.RETRY_STATE_IN_PROGRESS
is used internally only but is exposed via public api: https://github.com/temporalio/api/blob/master/temporal/api/enums/v1/workflow.proto#L107

This is causing some confusions to our users. If possible please remove it (keep it internal only).

Idk if this should be issue opened on the server or here. Please let me know if this repo is wrong place for it.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The enum declaration is in temporal/api/enums/v1/workflow.proto around line 107. Start by reviewing RetryState.RETRY_STATE_IN_PROGRESS and the API compatibility expectations; done means this internally used state is no longer exposed through the public API.

Written by the indexing model from the issue text.

Assessment

Tech stack
grpc
Domain
api
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.