Closed federicobond closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 97.41%. Comparing base (
c3ad697
) to head (3427796
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The _client_handlers dictionary allowed modifications during iteration without proper concurrency control. I added some reentrant locks to manage concurrent access to the _global_handlers and _client_handlers data structures.
See #326
Perhaps there is a way to implement some finer-grained concurrency control? The Java SDK uses a ConcurrentHashMap that allows retrieval operations without locking, but I haven't looked into the details.