zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hanm <...@git.apache.org>
Subject [GitHub] zookeeper pull request #561: ZOOKEEPER-3083: Remove some redundant and noisy...
Date Sat, 07 Jul 2018 05:01:45 GMT
Github user hanm commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/561#discussion_r200802655
  
    --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
    @@ -754,7 +754,7 @@ public void finishSessionInit(ServerCnxn cnxn, boolean valid) {
                 cnxn.sendBuffer(bb);
     
                 if (valid) {
    -                LOG.info("Established session 0x"
    +                LOG.debug("Established session 0x"
    --- End diff --
    
    This sounds pretty useful INFO level log. If we turn it into DEBUG level, are there any
other equivalent INFO level logging where we can derive the session establishment details?
If not, probably better to leave this as INFO?


---

Mime
View raw message