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

Consider inlining the Message Queue #541

Closed
kayabaNerve opened this issue Mar 11, 2024 · 1 comment
Closed

Consider inlining the Message Queue #541

kayabaNerve opened this issue Mar 11, 2024 · 1 comment
Labels
discussion This requires discussion improvement This could be better

Comments

@kayabaNerve
Copy link
Member

While documenting the Message Queue, I realized I'm not entirely sure why it exists, however ridiculous that may sound.

All of the design goals are sane. It's just, instead of using a remote server as the message buffer, we could use an async task on each side. The processor saves messages to be sent to its local database, and defines an async task to forward them along to the coordinator/receive messages from the coordinator. The coordinator would have a complimentary task.

This would actually be even-more robust, as technically, as of right now, the message queue can go offline and halt the processor/coordinator (though this is unexpected).

@kayabaNerve kayabaNerve added improvement This could be better discussion This requires discussion labels Mar 11, 2024
@kayabaNerve
Copy link
Member Author

The direction I want to go in for #565 supersedes this, even if this is currently valid.

@kayabaNerve kayabaNerve closed this as not planned Won't fix, can't repro, duplicate, stale May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion This requires discussion improvement This could be better
Projects
None yet
Development

No branches or pull requests

1 participant