You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm moving to go mod #7922
Maybe I can update logrus after that.
I know update logrus and support module are two different things, but update them once maybe save some time for me. @siddontang
Bug Report
In our test, we find there are too many mutex used
This is cause by the logrus
GetLevel
function which uses a global mutex. In the latest logrus, it has already switched to using atomic.I write a simple benchmark https://gist.github.com/siddontang/ee4cee6115b9d8d16286543c734f04d4 and find the atomic has a better performance than mutex.
Luckily, the latest logrus has already used atomic, we can update it directly.
The text was updated successfully, but these errors were encountered: