Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow reset API to bring new signals as atomic operation #553

Open
longquanzheng opened this issue Oct 25, 2024 · 2 comments
Open

allow reset API to bring new signals as atomic operation #553

longquanzheng opened this issue Oct 25, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@longquanzheng
Copy link

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

It would be really nice to allow reset API to bring a [few] new signals as atomic operation.
I can see several use cases for that
reset and then skip some activity that we don’t want to rerun. We could add some simple code to skip based on the signal. This will be much simpler than any other alternatives that I have seen.
reset with cherry pick some signals but not all. Currently we have the option to preserve all signals after reset point, or none. But it’s a hard decision when we want to preserve some.
Reset with some special behavior— just let the workflow knows that current run is in a reset state so that it can adjust the behavior accordingly.
Etc etc

Additional context

@longquanzheng longquanzheng added the enhancement New feature or request label Oct 25, 2024
@cretz
Copy link
Member

cretz commented Oct 28, 2024

This may be similar to #1203

@longquanzheng
Copy link
Author

This may be similar to #1203

Yeah that looks very similar: temporalio/temporal#1203

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants