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.



---

Reply via email to