zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bothejjms <...@git.apache.org>
Subject [GitHub] zookeeper pull request #563: ZOOKEEPER-3072: Throttle race condition fix
Date Fri, 27 Jul 2018 13:48:45 GMT
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.


---

Mime
View raw message