harmony-development / harmony-development/scherzo

perf: consider replacing broadcast channel with something else

Open
#107 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
29
Forks
2
PR merge metrics
No merged PRs in 30d

Description

A `dashmap::DashMap>` where `UserId` is the `u64` user id, `StreamId` is a `u8`. could probably be more memory efficient and faster (?) than the current solution (which is just a tokio broadcast channel + arced events). EventSender should be an unbounded sender so that we hold the lock for the least amount of time (smallmap also helps here as it is as fast as one can get with u8 keys)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.