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/06/05 21:15:36 UTC
svn commit: r1683848 - in /zookeeper/trunk: CHANGES.txt
src/java/main/org/apache/zookeeper/server/quorum/Follower.java
src/java/main/org/apache/zookeeper/server/quorum/Learner.java
src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java
Author: rgs
Date: Fri Jun 5 19:15:36 2015
New Revision: 1683848
URL: http://svn.apache.org/r1683848
Log:
ZOOKEEPER-2207: Enhance error logs with LearnerHandler.packetToString()
(Hitoshi Mitake via rgs)
Modified:
zookeeper/trunk/CHANGES.txt
zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java
zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java
zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java
Modified: zookeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/trunk/CHANGES.txt?rev=1683848&r1=1683847&r2=1683848&view=diff
==============================================================================
--- zookeeper/trunk/CHANGES.txt (original)
+++ zookeeper/trunk/CHANGES.txt Fri Jun 5 19:15:36 2015
@@ -187,6 +187,9 @@ IMPROVEMENTS:
ZOOKEEPER-2206: Add missing packet types to LearnerHandler.packetToString()
(Hitoshi Mitake via rgs)
+ ZOOKEEPER-2207: Enhance error logs with LearnerHandler.packetToString()
+ (Hitoshi Mitake via rgs)
+
Release 3.5.0 - 8/4/2014
NEW FEATURES:
Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java?rev=1683848&r1=1683847&r2=1683848&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java Fri Jun 5 19:15:36 2015
@@ -165,7 +165,7 @@ public class Follower extends Learner{
fzk.sync();
break;
default:
- LOG.warn("unknown type " + qp.getType());
+ LOG.warn("Unknown packet type: {}", LearnerHandler.packetToString(qp));
break;
}
}
Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java?rev=1683848&r1=1683847&r2=1683848&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java Fri Jun 5 19:15:36 2015
@@ -389,8 +389,8 @@ public class Learner {
}
else {
- LOG.error("Got unexpected packet from leader "
- + qp.getType() + " exiting ... " );
+ LOG.error("Got unexpected packet from leader: {}, exiting ... ",
+ LearnerHandler.packetToString(qp));
System.exit(13);
}
Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java?rev=1683848&r1=1683847&r2=1683848&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java Fri Jun 5 19:15:36 2015
@@ -481,7 +481,8 @@ public class LearnerHandler extends ZooK
qp = new QuorumPacket();
ia.readRecord(qp, "packet");
if(qp.getType() != Leader.ACK){
- LOG.error("Next packet was supposed to be an ACK");
+ LOG.error("Next packet was supposed to be an ACK,"
+ + " but received packet: {}", packetToString(qp));
return;
}