[GitHub] zookeeper pull request #453: ZOOKEEPER-2845: Apply commit log when restartin...

[afine]

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

    https://github.com/apache/zookeeper/pull/453#discussion_r168649459
  
    --- Diff: 
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
    @@ -888,4 +888,127 @@ public void testWithOnlyMinSessionTimeout() throws 
Exception {
                     maxSessionTimeOut, quorumPeer.getMaxSessionTimeout());
         }
     
    +    @Test
    +    public void testTxnAheadSnapInRetainDB() throws Exception {
    --- End diff --
    
    nit: I don't think we use the terminology "RetainDB" anywhere else. Perhaps 
we can get rid of "retain"?


---