先来一段简单的测试代码:

package javaapplication23;

import java.io.IOException;
import java.util.logging.FileHandler;

public class JavaApplication23 {
    public static void main(String[] args) throws IOException {
        new FileHandler("./test_%u_%g.log", 10000, 100, true);
    }
}

此测试代码使用 Java 7 仅创建一个文件“test_0_0.log”，无论我运行该程序的频率如何。这是预期的行为，因为构造函数中的附加参数设置为 true。

但是如果我在 Java 8 中运行这个示例，每次运行都会创建一个新文件(test_0_0.log、test_0_1.log、test_0_2.log，...)。我认为这是一个错误。

恕我直言，Java 中的相关更改是:

@@ -413,18 +428,18 @@
                     // object.  Try again.
                     continue;
                 }
-                FileChannel fc;
+
                 try {
-                    lockStream = new FileOutputStream(lockFileName);
-                    fc = lockStream.getChannel();
-                } catch (IOException ix) {
-                    // We got an IOException while trying to open the file.
-                    // Try the next file.
+                    lockFileChannel = FileChannel.open(Paths.get(lockFileName),
+                            CREATE_NEW, WRITE);
+                } catch (FileAlreadyExistsException ix) {
+                    // try the next lock file name in the sequence
                     continue;
                 }
+
                 boolean available;
                 try {
-                    available = fc.tryLock() != null;
+                    available = lockFileChannel.tryLock() != null;
                     // We got the lock OK.
                 } catch (IOException ix) {
                     // We got an IOException while trying to get the lock.
@@ -440,7 +455,7 @@
                 }

                 // We failed to get the lock.  Try next file.
-                fc.close();
+                lockFileChannel.close();
             }
         }

(完整:OpenJDK changeset 6123:ac22a52a732c)

我知道通常 FileHandler 会被 Logmanager 关闭，但如果系统或应用程序崩溃或进程被终止，情况就不是这样了。这就是为什么我在上面的示例代码中没有“关闭”语句的原因。

现在我有两个问题:

1) 你有什么看法？这是一个错误吗？ (几乎在下面的评论和答案中回答)

2) 您知道在 Java 8 中获得旧 Java 7 行为的解决方法吗？ (更重要的问题...)

感谢您的回答。

最佳答案

关闭 FileHandler 会删除“lck”文件。如果锁定文件完全存在于 JDK8 version that is less than update 40 (java.util.logging) 下，FileHandler 将旋转。来自OpenJDK讨论中，除了当前进程无法锁定它之外，如果 lck 文件存在，则决定始终轮换。给出的理由是当锁文件存在时旋转总是更安全。因此，如果您将旋转模式与 JDK 版本混合使用，这将变得非常糟糕，因为 JDK7 版本将重用锁，但 JDK8 版本将保留它并旋转。这就是您对测试用例所做的。

如果我从工作目录清除所有日志和 lck 文件然后运行，则使用 JDK8:

public static void main(String[] args) throws IOException {
    System.out.println(System.getProperty("java.runtime.version"));
    new FileHandler("./test_%u.log", 10000, 100, true).close();
}

我总是看到名为“test_0.log.0”的文件。我使用 JDK7 得到了相同的结果。

底线是您必须确保您的 FileHandlers 已关闭。如果它永远不会garbaged collected或者从记录器树中删除，然后 LogManager 将关闭您的 FileHandler。否则你必须关闭它。修复后，在运行新的修补代码之前清除所有锁定文件。然后请注意，如果 JVM 进程崩溃或被终止，锁定文件将不会被删除。如果您在关闭时遇到 I/O 错误，您的锁定文件将不会被删除。当下一个进程开始时，FileHandler 将轮换。

正如您所指出的，如果上述情况发生超过 100 次，则可能会用完 JDK8 上的所有锁定文件。对此的一个简单测试是在不删除日志和 lck 文件的情况下运行以下代码两次:

public static void main(String[] args) throws Exception {
    System.out.println(System.getProperty("java.runtime.version"));
    ReferenceQueue<FileHandler> q = new ReferenceQueue<>();
    for (int i=0; i<100; i++) {
        WeakReference<FileHandler> h = new WeakReference<>(
                new FileHandler("./test_%u.log", 10000, 2, true), q);
        while (q.poll() != h) {
            System.runFinalization();
            System.gc();
            System.runFinalization();
            Thread.yield();
        }
    }
}

但是，如果JDK-6774110，上面的测试用例将无法工作。已正确修复。可以在 OpenJDK 网站上的 RFR: 8048020 - Regression on java.util.logging.FileHandler 下跟踪此问题。和 FileHandler webrev .

关于java - Java 8 中的 java.util.logging.FileHandler 损坏了吗？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/24321098/