Github user lvfangmin commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/629#discussion_r218676806
  
    --- Diff: src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java ---
    @@ -819,7 +819,7 @@ public void commandRun() {
                 }
             }
     
    -        private void print(String key, long number) {
    +        private void print(String key, Object number) {
    --- End diff --
    
    It's better to override print with double number, which will be more 
efficient, as we don't need to cast all the numbers to Object while printing 
the metrics.


---

Reply via email to