zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From karanmehta93 <...@git.apache.org>
Subject [GitHub] zookeeper issue #307: ZOOKEEPER-2770 ZooKeeper slow operation log
Date Sat, 15 Jul 2017 23:21:16 GMT
Github user karanmehta93 commented on the issue:

    https://github.com/apache/zookeeper/pull/307
  
    @hanm @eribeiro 
    Thank you for your comments. I have added a rebased patch with a test in `ZooKeeperServerMainTest`
class. Please review whenever convenient. 
    
    > It's odd to have two (or in future even more) types of latency checks scattered around
which creates fragmentation w.r.t. the definition of what a request latency means. 
    
    We can decide based on opinion from other people and I can update the patch accordingly.
I will create new JIRA's based on @tdunning suggestions, after this one gets approved. I am
eager to take them up.
    
    Documentation is still pending. Will add a patch for it soon. I am not sure if I have
to modify the `ZooKeeperAdmin.xml` file or `ZooKeeperAdmin.html` file. Can you point me to
any relevant stuff?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message