[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-02-16 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r168891236 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -35,92 +38,181 @@ public class

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-02-14 Thread eolivelli
Github user eolivelli commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r168301786 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -35,92 +38,181 @@ public class

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-02-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zookeeper/pull/450 ---

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-02-01 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r165525848 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164606812 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605998 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605964 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605860 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605834 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605909 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605723 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -294,5 +297,25 @@ public int compare(File o1, File o2) {

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605619 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -294,5 +297,25 @@ public int compare(File o1, File o2) {

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605521 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-29 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r164605358 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread afine
Github user afine commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163395510 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -50,6 +50,9 @@ private static final String SNAP_DIR="snapDir";

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread afine
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

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread afine
Github user afine commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163472195 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader hdr,

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163457880 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -83,7 +86,7 @@ public static URI makeFileLoggerURL(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163457798 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163290319 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -294,5 +297,25 @@ public int compare(File o1, File o2) {

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163290119 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -294,5 +297,25 @@ public int compare(File o1, File o2) {

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163289284 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -83,7 +86,7 @@ public static URI makeFileLoggerURL(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163282674 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-23 Thread mfenes
Github user mfenes commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r163282090 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162946062 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162947080 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162929129 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162945589 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -294,5 +297,25 @@ public int compare(File o1, File o2) {

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162945403 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -83,7 +86,7 @@ public static URI makeFileLoggerURL(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162946298 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162945532 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/Util.java --- @@ -294,5 +297,25 @@ public int compare(File o1, File o2) {

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162948359 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162947464 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162954011 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162929390 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java --- @@ -136,13 +136,51 @@ public FileTxnSnapLog(File dataDir, File

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-22 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/450#discussion_r162946041 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/FileTxnSnapLogTest.java --- @@ -159,4 +159,222 @@ public void onTxnLoaded(TxnHeader

[GitHub] zookeeper pull request #450: ZOOKEEPER-2967: Add check to validate dataDir a...

2018-01-19 Thread mfenes
GitHub user mfenes opened a pull request: https://github.com/apache/zookeeper/pull/450 ZOOKEEPER-2967: Add check to validate dataDir and dataLogDir paramete… ZOOKEEPER-2967: Add check to validate dataDir and dataLogDir parameters at startup This PR adds a check to