Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use concurrent map implementation #1318

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Use concurrent map implementation #1318

merged 1 commit into from
Jan 8, 2025

Conversation

anderseknert
Copy link
Member

To avoid mutex handling cluttering up the code, use a concurrent wrapper for maps where we need it.

To avoid mutex handling cluttering up the code, use a concurrent
wrapper for maps where we need it.

Signed-off-by: Anders Eknert <anders@styra.com>
@anderseknert anderseknert marked this pull request as ready for review January 8, 2025 10:06
Copy link
Member

@charlieegan3 charlieegan3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 this is great!

@anderseknert anderseknert merged commit 34f9827 into main Jan 8, 2025
5 checks passed
@anderseknert anderseknert deleted the concurrent-map branch January 8, 2025 11:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants