zookeeper-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From si...@apache.org
Subject svn commit: r1465437 - in /zookeeper/bookkeeper/trunk: CHANGES.txt bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java
Date Sun, 07 Apr 2013 18:07:06 GMT
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



Mime
View raw message