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.

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

add mutex to prevent concurrent map access #428

Conversation

yun-yeo
Copy link
Contributor

@yun-yeo yun-yeo commented Aug 30, 2021

close #427

  • changed nodedb's mutex to RWMutex to allow Muti-Thread access

@yun-yeo yun-yeo force-pushed the bugfix/mutex-to-prevent-concurrent-map-access branch 2 times, most recently from c35da87 to e2641d1 Compare August 30, 2021 07:35
@tac0turtle
Copy link
Member

@odeke-em is bencher enabled on this repo?

@odeke-em
Copy link
Contributor

@odeke-em is bencher enabled on this repo?

No it isn't enabled. Let me message you privately to give you the credentials to enable it via GitHub webhook settings.

@robert-zaremba robert-zaremba added the S:automerge Automatic merge and/or update Pull requests label Aug 30, 2021
@robert-zaremba robert-zaremba merged commit 08c4ae4 into cosmos:master Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S:automerge Automatic merge and/or update Pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] concurrent map read and map write
4 participants