[GitHub] zookeeper pull request #563: ZOOKEEPER-3072: Throttle race condition fix

[bothejjms]

Github user bothejjms commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/563#discussion_r205778339
  
    --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
    @@ -1128,9 +1128,9 @@ public void processPacket(ServerCnxn cnxn, ByteBuffer 
incomingBuffer) throws IOE
                     Record rsp = processSasl(incomingBuffer,cnxn);
                     ReplyHeader rh = new ReplyHeader(h.getXid(), 0, 
KeeperException.Code.OK.intValue());
                     cnxn.sendResponse(rh,rsp, "response"); // not sure about 
3rd arg..what is it?
    -                return;
    --- End diff --
    
    I have refactored like that.
    Returns are actually unnecessary but I have consistently added them now.


---