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

Provide a faster / cleaner path for "mandatory" Messages #45

Open
gbin opened this issue Sep 21, 2024 · 0 comments
Open

Provide a faster / cleaner path for "mandatory" Messages #45

gbin opened this issue Sep 21, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@gbin
Copy link
Collaborator

gbin commented Sep 21, 2024

We changed the API to allow the tasks to send Optional Payloads in messages.
This is important for example when you have merging tasks that might not have a 1 to 1 matching messages from 2 different sources.

But it added the burden and a little overhead for all task to have to check the Option<> for all payloads even if by construct they might not need to be optional.

Idea: make another type of messages that are like the old ones (with no Option<>) and provide an option in the macros and the DAG to use them.

@gbin gbin added the enhancement New feature or request label Dec 5, 2024
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

1 participant