search

go-graphite / carbonzipper

proxy to transparently merge graphite carbon backends
Other
104 stars 29 forks source link

Use non-format error function #67

Closed gunnihinn closed 6 years ago

gunnihinn commented 6 years ago

Calling

errors.Fatalf("timeout")

makes for a lot of

{"error":"timeout%!(EXTRA []interface {}=[])"}

lines in the log.

dgryski commented 6 years ago

Arguably this is a bug in the errors package too.

gunnihinn commented 6 years ago

Agreed. It can figure this situation out by itself.

On Wed, 27 Jun 2018 at 05:12, Damian Gryski notifications@github.com wrote:

Arguably this is a bug in the errors package too.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/go-graphite/carbonzipper/pull/67#issuecomment-400529343, or mute the thread https://github.com/notifications/unsubscribe-auth/AF1xtlQZN1gvaTEKnmUyPrBuRhS_qLrdks5uAvgYgaJpZM4U4IoI .

-- GNUPG key id: 0x3273D9DA