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

Add reply_mode to command meta data #425

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

kjnilsson
Copy link
Contributor

@kjnilsson kjnilsson commented Mar 8, 2024

This would allow state machines to make use of, e.g. the pid in the {notify, Corr, Pid} reply mode to avoid having to include Pids in the command body and thus avoid them being stored twice for each command.

This would allow state machines to make use of, e.g. the pid in
the {notify, Corr, Pid} reply mode to avoid having to include
Pids in the command body and thus avoid them being stored twice
for each command.
Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great idea!

@kjnilsson kjnilsson merged commit 57da222 into main Mar 8, 2024
7 checks passed
@dumbbell dumbbell deleted the add-reply-mode-to-command-meta-data branch March 8, 2024 14:19
@kjnilsson kjnilsson added this to the 2.10.0 milestone Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants