Github user lujiefsi commented on the issue:
https://github.com/apache/zookeeper/pull/495
@phunt :
I think the reason that @brettKK use RuntimeException to replace
NullPointerException is :(1)NullPointerException is subclass of
RuntimeException (2) give the semantic reason instead ugly NPE.(3)@brettkk may
dost not think over what happens when the RTE is thrown. He/She may just do it
due to (2) .---
