You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by mi...@apache.org on 2013/10/05 00:17:00 UTC

svn commit: r1529330 - in /zookeeper/trunk: CHANGES.txt src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java src/java/systest/org/apache/zookeeper/test/system/QuorumPeerInstance.java

Author: michim
Date: Fri Oct  4 22:17:00 2013
New Revision: 1529330

URL: http://svn.apache.org/r1529330
Log:
ZOOKEEPER-1766 Consistent log severity level guards and statements (Jackie Chang via michim)

Modified:
    zookeeper/trunk/CHANGES.txt
    zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java
    zookeeper/trunk/src/java/systest/org/apache/zookeeper/test/system/QuorumPeerInstance.java

Modified: zookeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/trunk/CHANGES.txt?rev=1529330&r1=1529329&r2=1529330&view=diff
==============================================================================
--- zookeeper/trunk/CHANGES.txt (original)
+++ zookeeper/trunk/CHANGES.txt Fri Oct  4 22:17:00 2013
@@ -421,6 +421,9 @@ BUGFIXES:
   ZOOKEEPER-732. Improper translation of error into Python exception
   (Andrei Savu, Lei Zhang, fpj via fpj)
 
+  ZOOKEEPER-1766 Consistent log severity level guards and statements
+  (Jackie Chang via michim)
+
 IMPROVEMENTS:
 
   ZOOKEEPER-1170. Fix compiler (eclipse) warnings: unused imports,

Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java?rev=1529330&r1=1529329&r2=1529330&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java Fri Oct  4 22:17:00 2013
@@ -153,7 +153,7 @@ public class SessionTrackerImpl extends 
 
     synchronized public void setSessionClosing(long sessionId) {
         if (LOG.isTraceEnabled()) {
-            LOG.info("Session closing: 0x" + Long.toHexString(sessionId));
+            LOG.trace("Session closing: 0x" + Long.toHexString(sessionId));
         }
         SessionImpl s = sessionsById.get(sessionId);
         if (s == null) {

Modified: zookeeper/trunk/src/java/systest/org/apache/zookeeper/test/system/QuorumPeerInstance.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/systest/org/apache/zookeeper/test/system/QuorumPeerInstance.java?rev=1529330&r1=1529329&r2=1529330&view=diff
==============================================================================
--- zookeeper/trunk/src/java/systest/org/apache/zookeeper/test/system/QuorumPeerInstance.java (original)
+++ zookeeper/trunk/src/java/systest/org/apache/zookeeper/test/system/QuorumPeerInstance.java Fri Oct  4 22:17:00 2013
@@ -88,7 +88,7 @@ class QuorumPeerInstance implements Inst
             // us which machine we are
             serverId = Integer.parseInt(parts[0]);
             if (LOG.isDebugEnabled()) {
-                LOG.info("Setting up server " + serverId);
+                LOG.debug("Setting up server " + serverId);
             }
             if (parts.length > 1 && parts[1].equals("false")) {
                 System.setProperty("zookeeper.leaderServes", "no");