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

fix race in zk library #250

Closed jhump closed 4 years ago

jhump commented 4 years ago

Is this library still active? Should I try to land this fix? Or should I instead pursue https://github.com/go-zookeeper/zk/pull/35 and close this one?

jhump commented 4 years ago

@samuel, can you take a look and let me know what I should do with this?

coveralls commented 4 years ago

Coverage Status

Coverage increased (+0.3%) to 79.87% when pulling bb98ea77bfd510bc3537fd0276945c1a5850fe38 on jhump:jh/race2 into 2cc03de413da42869e2db7ce7965d3e978d917eb on samuel:master.

jhump commented 4 years ago

Based on response in #249, I'll pursue my PR in the fork of this repo.