zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bitgaoshu <...@git.apache.org>
Subject [GitHub] zookeeper pull request #356: ZOOKEEPER-2572: Fix potential resource leak in ...
Date Thu, 07 Sep 2017 06:55:32 GMT
Github user bitgaoshu commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/356#discussion_r137459795
  
    --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java ---
    @@ -409,13 +409,15 @@ public void truncate(long zxid) throws IOException {
                     try {
                         Files.delete(itr.logFile.toPath());
                     } catch (NoSuchFileException e) {
    +                    LOG.info("An NoSuchFileException was thrown when delete file {}"
+
    +                    ", but will continue. Assume this file has been deleted successfully.",
itr.logFile);
                     }
                 }
             } finally {
    -            close(itr);
                 if (raf != null) {
                     raf.close();
    --- End diff --
    
    update


---

Mime
View raw message