Change logging to use uber zap

nspcc-dev / neo-go

Go Node and SDK for the Neo blockchain

MIT License

123 stars 79 forks source link

Change logging to use uber zap #284

Closed volekerb closed 4 years ago

volekerb commented 5 years ago

As proposed in #247 we have to change fmt.println logging to log but at the same time suggest to use uber/zap since it has better performance compare to logrus which is used in master

fyrchik commented 4 years ago

There are more important advantages imo:

zap can be disabled/enabled per component
zap can be hierarchical (one can create child loggers based on parent ones, all fields will be inherited)

roman-khimov commented 4 years ago

Now that the DBFT library is using it it's more important to fix this issue for logging consistency.