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.

Sign up for GitHub

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

Jump to bottom

refactor(processor): replace sync.Map with xsync.MapOf #135

Merged
merged 2 commits into from
Sep 7, 2024
Merged

refactor(processor): replace sync.Map with xsync.MapOf #135

merged 2 commits into from
Sep 7, 2024

Conversation

nuxencs
Copy link
Owner

@nuxencs nuxencs commented Sep 7, 2024 •
edited
Loading

Replace the standard library sync.Map with xsync.MapOf which performs much better and is a lot easier to use.
Also use the client name as the key for the clientMap and torrentMap, because it is already a unique key.

@nuxencs nuxencs added this to the v0.9.1 milestone Sep 7, 2024
@nuxencs nuxencs added Area: API Related to the API Type: Refactor Refactoring of code labels Sep 7, 2024
@nuxencs nuxencs marked this pull request as ready for review September 7, 2024 12:51
@nuxencs nuxencs merged commit 06fb139 into develop Sep 7, 2024
12 checks passed
@nuxencs nuxencs deleted the refactor/replace-sync-map-with-xsync branch September 7, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zze0s zze0s zze0s approved these changes

Assignees
No one assigned
Labels
Area: API Related to the API Type: Refactor Refactoring of code
Projects
None yet
Milestone
v0.10.0
Development

Successfully merging this pull request may close these issues.
2 participants
@nuxencs @zze0s