Skip to content

Conversation

jan-tennert
Copy link
Collaborator

@jan-tennert jan-tennert commented Sep 10, 2025

What kind of change does this PR introduce?

Improvement

What is the current behavior?

  • All callbacks (broadcast, presence, postgres) are stored in a single list

What is the new behavior?

  • Split callbacks in three different lists / maps.
    • For broadcast callback a map is used, a callback list is stored under the respective event name
    • For presence callbacks a map is used, callbacks are mapped by their callback id for easy removal
    • Same with postgres callbacks

@jan-tennert jan-tennert changed the title Remove AtomicFu, split callbacks in the CallbackManager Split callbacks in the CallbackManager Sep 10, 2025
@jan-tennert jan-tennert merged commit 7ff51e3 into master Sep 10, 2025
13 checks passed
@jan-tennert jan-tennert deleted the remove-atomicfu branch September 10, 2025 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant