-
Notifications
You must be signed in to change notification settings - Fork 142
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
Replace tokio mutex with std mutex #1062
Comments
Please feel free to open a PR! |
Whoops, didn't mean to close it. :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
IMHO, I suggest that replacing
tokio::sync::Mutex
withstd::sync::Mutex
can potentially lead to better performance outcomes. This assertion is based on the design differences between the two mutex implementations, particularly in asynchronous contexts.For further insights, check out the following resources:
Discussion on this topic. Never use Tokio mutexes.
tokio::sync::Mutex documentation.
The text was updated successfully, but these errors were encountered: