-
Notifications
You must be signed in to change notification settings - Fork 118
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
Switch out logrus as the logger #65
Comments
Two self-promotions here:
|
One other strategy I have done in the past: https://github.com/grpc/grpc-go/tree/master/grpclog |
I vote to stick with logrus - it does what we want it do - we use it's structured logging and is not in the i/o path. We can revisit this in the future if need be. |
@peter-edge No issues using dlog but can you submit a separate pull request for this (it's in refactor and refactor is very large to go through.) |
merged #111 |
it is very slow
The text was updated successfully, but these errors were encountered: