search

samuel / go-zookeeper

Native ZooKeeper client for Go. This project is no longer maintained. Please use https://github.com/go-zookeeper/zk instead.
BSD 3-Clause "New" or "Revised" License
1.64k stars 672 forks source link

Print EOF or nil error #199

Open wrfly opened 5 years ago

wrfly commented 5 years ago

https://github.com/samuel/go-zookeeper/blob/c4fab1ac1bec58281ad0667dc3f0907a9476ac47/zk/conn.go#L510-L512

https://github.com/samuel/go-zookeeper/blob/c4fab1ac1bec58281ad0667dc3f0907a9476ac47/zk/conn.go#L525-L527

I don't think it's OK to print the EOF error or nil error only because the logInfo flag, since the message won't help.