zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hanm <...@git.apache.org>
Subject [GitHub] zookeeper issue #628: ZOOKEEPER-3140: Allow Followers to host Observers
Date Tue, 23 Oct 2018 20:47:47 GMT
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.



---

Mime
View raw message