Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/567#discussion_r202364057
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java ---
@@ -347,6 +397,13 @@ public synchronized void commit() throws IOException {
while (streamsToFlush.size() > 1) {
streamsToFlush.removeFirst().close();
}
+
+ // Roll the log file if we exceed the size limit
+ long logSize = getCurrentLogSize();
+ if ((logSizeLimit > 0) && (logSize > logSizeLimit)) {
--- End diff --
Redundant inner brackets.
---
