zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eolivelli <...@git.apache.org>
Subject [GitHub] zookeeper pull request #622: [ZOOKEEPER-3145] Fix potential watch missing is...
Date Fri, 14 Sep 2018 12:03:17 GMT
Github user eolivelli commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/622#discussion_r217689355
  
    --- Diff: src/java/main/org/apache/zookeeper/server/util/SerializeUtils.java ---
    @@ -116,6 +120,10 @@ public static Record deserializeTxn(byte txnBytes[], TxnHeader hdr)
                         create.setAcl(createv0.getAcl());
                         create.setEphemeral(createv0.getEphemeral());
                         create.setParentCVersion(-1);
    +                } else if (hdr.getType() == OpCode.closeSession) {
    +                    // perhaps this is before CloseSessionTxn was added,
    --- End diff --
    
    That was my guess, but I just wanted a confirmation. Thanks


---

Mime
View raw message