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

    https://github.com/apache/zookeeper/pull/450#discussion_r163395848
  
    --- Diff: 
src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java ---
    @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File snapDir) 
throws IOException {
                 throw new DatadirException("Cannot write to snap directory " + 
this.snapDir);
             }
     
    +        // check content of transaction log and snapshot dirs if they are 
two different directories
    --- End diff --
    
    Could we add a reference to the discussion that explains why this whole 
check is needed in the comments and/or the exception that the user sees?


---

Reply via email to