Author: sijie
Date: Sun Apr 7 18:07:06 2013
New Revision: 1465437
URL: http://svn.apache.org/r1465437
Log:
BOOKKEEPER-599: NPE in PerChannelBookieClient (jiannan via sijie)
Modified:
zookeeper/bookkeeper/trunk/CHANGES.txt
zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java
Modified: zookeeper/bookkeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/bookkeeper/trunk/CHANGES.txt?rev=1465437&r1=1465436&r2=1465437&view=diff
==============================================================================
--- zookeeper/bookkeeper/trunk/CHANGES.txt (original)
+++ zookeeper/bookkeeper/trunk/CHANGES.txt Sun Apr 7 18:07:06 2013
@@ -42,6 +42,8 @@ Trunk (unreleased changes)
BOOKKEEPER-559: Fix occasional failure in AuditorBookieTest (ivank via umamahesh)
+ BOOKKEEPER-599: NPE in PerChannelBookieClient (jiannan via sijie)
+
IMPROVEMENTS:
BOOKKEEPER-555: Make BookieServer use Netty rather than a custom IO server (ivank)
Modified: zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java
URL: http://svn.apache.org/viewvc/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java?rev=1465437&r1=1465436&r2=1465437&view=diff
==============================================================================
--- zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java
(original)
+++ zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java
Sun Apr 7 18:07:06 2013
@@ -569,8 +569,8 @@ public class PerChannelBookieClient exte
void handleReadResponse(BookieProtocol.ReadResponse rr) {
if (LOG.isDebugEnabled()) {
- LOG.debug("Got response for read request {} entry length: {}",
- rr, rr.getData().readableBytes());
+ LOG.debug("Got response for read request {} entry length: {}", rr,
+ rr.getData() != null ? rr.getData().readableBytes() : -1);
}
// convert to BKException code because thats what the uppper
|