Skip to content

Actor framework and MessageBus v3 #2402

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

Merged
merged 14 commits into from
Mar 6, 2025
Merged

Actor framework and MessageBus v3 #2402

merged 14 commits into from
Mar 6, 2025

Conversation

twitu
Copy link
Collaborator

@twitu twitu commented Mar 4, 2025

Pull Request

Global static variables for actor registry and message bus with helper functions to access and set state. Actor state is wrapped using unsafe cell to conveniently get mutable access. Related #1711

Implement throttler using actor model and message handlers.

Type of change

  • New feature (non-breaking change which adds functionality)

How has this change been tested?

Throttler tests passing.

twitu added 6 commits March 3, 2025 22:28
Define helper functions to access and set state
Use message bus send style mental model
Unsafe cell to access actor mutable state
Intuitive definition for callbacks
Use nightly fn traits to express recursive
throttler process callback
@twitu twitu closed this Mar 4, 2025
twitu added 4 commits March 6, 2025 13:26
Register throttler process buffer messages
as a handler on the message bus and call
it indirectly via message bus to untie the
recursive definition
@twitu twitu reopened this Mar 6, 2025
@twitu twitu marked this pull request as ready for review March 6, 2025 18:35
@cjdsellers cjdsellers changed the title Actor framework and Message Bus V3 Actor framework and MessageBus v3 Mar 6, 2025
@cjdsellers cjdsellers merged commit f475a73 into develop Mar 6, 2025
12 checks passed
@cjdsellers cjdsellers deleted the message-bus-v3 branch March 6, 2025 20:59
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