ZkConnection not really thread-safe

[facboy]

ZkConnection does not really look thread-safe to me. It locks _zk for the connect and close methods, but _zk itself is not volatile so there are no visibility guarantees when it is accessed by all the other methods.

as an aside, ReentrantLock could easily be replaced by a synchronized block/method in connect and close.