-
Notifications
You must be signed in to change notification settings - Fork 28
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
Orchestration ID reuse policies #51
Conversation
create alias for protos type update test move duplicate code outside backend implementation update paramter name
tag @ItalyPaleAle , @RyanLettieri for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few questions from me:
76c6eeb
to
fde7991
Compare
simplify status check update tests clean up fix test
fde7991
to
12222af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Glad to have this change! Let's wait to merge until the Dapr PR which uses it is also signed off.
This PR is a follow-up PR for #46. It include some updates to support reuse ID policy in dapr workflow
Related issue #42