zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a470577391 <...@git.apache.org>
Subject [GitHub] zookeeper pull request #553: continue can be replaced with break
Date Thu, 28 Jun 2018 09:55:29 GMT
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.


---

Mime
View raw message