Github user a470577391 commented on the issue:
https://github.com/apache/zookeeper/pull/573
@maoling It has been reopened.
---
GitHub user a470577391 reopened a pull request:
https://github.com/apache/zookeeper/pull/573
ZOOKEEPER-3090 continue can be replaced with break
change continue to break
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/a470577391
Github user a470577391 closed the pull request at:
https://github.com/apache/zookeeper/pull/573
---
GitHub user a470577391 opened a pull request:
https://github.com/apache/zookeeper/pull/573
ZOOKEEPER-3090 continue can be replaced with break
change continue to break
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/a470577391
Github user a470577391 closed the pull request at:
https://github.com/apache/zookeeper/pull/553
---
Github user a470577391 commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/553#discussion_r198817324
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java ---
@@ -254,7 +254,7 @@ public synchronized boolean append(TxnHeader
Github user a470577391 commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/553#discussion_r198784188
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java ---
@@ -254,7 +254,7 @@ public synchronized boolean append(TxnHeader
GitHub user a470577391 opened a pull request:
https://github.com/apache/zookeeper/pull/553
continue can be replaced with break
FileTxnLog.getLogFiles()
continue can be replaced with break
You can merge this pull request into a Git repository by running:
$ git pull https