[GitHub] zookeeper pull request #553: continue can be replaced with break

[a470577391]

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 hdr, 
Record txn)
             for (File f : files) {
                 long fzxid = Util.getZxidFromName(f.getName(), 
LOG_FILE_PREFIX);
                 if (fzxid > snapshotZxid) {
    -                continue;
    +                break;
                 }
                 // the files
    --- End diff --
    
    Thank you for your suggestion.
    1. i'm going to try.
    2. i think use Java8 may be unfriendly to the lower version.


---