Github user hanm commented on the issue:

    https://github.com/apache/zookeeper/pull/580
  
    From the conversation so far, it looks like we have reached a consensus 
regarding the directions of this patch and future metrics related patch. For 
this specific patch, it looks good to me, and I only have one comment regarding 
FOLLOWER_SYNC_TIME instrumentation but that's not blocking.
    
    Any comments from other reviewers? @anmolnar @eolivelli @lvfangmin 
    Let's try to get this patch merge in soon to avoid constant turnovers on 
merge conflicts (unfortunately, it got another merge conflicts that requires 
another rebase.).


---

Reply via email to