Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/628 >> This patch does not touch the c client or the default configurations for those tests so I'm unsure how to proceed. My feeling is the failure is a flaky test, and has nothing to do with this patch. Though, it would be good if we can identify the exact failing test case, and rule out the possibility that it's caused by this patch (since C client depends on same java server code.). Also, sorry for lagging on following up my previous review. I am resuming reviewing this patch this week.
---