removing/disabling logging about go-zookeeper HOT 2 CLOSED

I believe there's 3 instance of logging in the server: failed to connect to server, failed to set previous watches, and a catch all for unexpected errors (which likely point to bigger issues).

One alternative is to turn off logging and use exported metrics so that any existing alerting infrastructure can be used (e.g. failed to connect N times in M seconds).. this likely should be added anyway regardless of logging.

Do you think it would be ok if you could pass in a custom logger to the client (func (c *Conn) SetLogger(Logger)) conforming to an interface like:

type Logger interface {
    Printf(format string, v ...interface{})
}

and by default the stdlib log would be used. Then could override it to disable logging entirely or reroute to a more appropriate place (perhaps syslog if using it).

from go-zookeeper.

I think that would make sense to me. Then have an unexported struct field for the Logger. To avoid breaking the old way, we could default to setting up a logger that has the same behavior as now.

from go-zookeeper.