You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by rg...@apache.org on 2015/11/09 00:05:49 UTC
svn commit: r1713312 - in /zookeeper/trunk: CHANGES.txt
src/java/main/org/apache/zookeeper/ClientCnxn.java
Author: rgs
Date: Sun Nov 8 23:05:49 2015
New Revision: 1713312
URL: http://svn.apache.org/viewvc?rev=1713312&view=rev
Log:
ZOOKEEPER-2315: Change client connect zk service timeout log level from Info
to Warn level (Lin Yiqun via rgs)
Modified:
zookeeper/trunk/CHANGES.txt
zookeeper/trunk/src/java/main/org/apache/zookeeper/ClientCnxn.java
Modified: zookeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/trunk/CHANGES.txt?rev=1713312&r1=1713311&r2=1713312&view=diff
==============================================================================
--- zookeeper/trunk/CHANGES.txt (original)
+++ zookeeper/trunk/CHANGES.txt Sun Nov 8 23:05:49 2015
@@ -297,6 +297,9 @@ IMPROVEMENTS:
ZOOKEEPER-2040: Server to log underlying cause of SASL connection problems.
(Steve Loughran via cnauroth)
+ ZOOKEEPER-2315: Change client connect zk service timeout log level from Info
+ to Warn level (Lin Yiqun via rgs)
+
Release 3.5.0 - 8/4/2014
NEW FEATURES:
Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/ClientCnxn.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/ClientCnxn.java?rev=1713312&r1=1713311&r2=1713312&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/ClientCnxn.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/ClientCnxn.java Sun Nov 8 23:05:49 2015
@@ -1197,11 +1197,14 @@ public class ClientCnxn {
}
if (to <= 0) {
- throw new SessionTimeoutException(
- "Client session timed out, have not heard from server in "
- + clientCnxnSocket.getIdleRecv() + "ms"
- + " for sessionid 0x"
- + Long.toHexString(sessionId));
+ String warnInfo;
+ warnInfo = "Client session timed out, have not heard from server in "
+ + clientCnxnSocket.getIdleRecv()
+ + "ms"
+ + " for sessionid 0x"
+ + Long.toHexString(sessionId);
+ LOG.warn(warnInfo);
+ throw new SessionTimeoutException(warnInfo);
}
if (state.isConnected()) {
//1000(1 second) is to prevent race condition missing to send the second ping
@@ -1382,9 +1385,12 @@ public class ClientCnxn {
Watcher.Event.EventType.None,
Watcher.Event.KeeperState.Expired, null));
eventThread.queueEventOfDeath();
- throw new SessionExpiredException(
- "Unable to reconnect to ZooKeeper service, session 0x"
- + Long.toHexString(sessionId) + " has expired");
+
+ String warnInfo;
+ warnInfo = "Unable to reconnect to ZooKeeper service, session 0x"
+ + Long.toHexString(sessionId) + " has expired";
+ LOG.warn(warnInfo);
+ throw new SessionExpiredException(warnInfo);
}
if (!readOnly && isRO) {
LOG.error("Read/write client got connected to read-only server");