RFR: 8352642: Set zipinfo-time=false when constructing zipfs FileSystem in com.sun.tools.javac.file.JavacFileManager$ArchiveContainer for better performance

Jason Zaugg jzaugg at openjdk.org
Mon Mar 24 22:36:07 UTC 2025


On Mon, 24 Mar 2025 19:20:27 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

> Ordinarily, I would say a test is required, but I can't think up a viable test to verify the effect of this change in a test, so we probably need to do it without a test.

I tested this change [externally](https://github.com/retronym/jarcache/blob/ba7f5cdd5f7db976a6568855a0d517f52a1404bf/demo/src/test/scala/demo/JarCacheAgentTest.java#L75-L76) with [JFRUnit](https://github.com/moditect/jfrunit). 

If preferred, I could create a `jtreg` test along similar lines. It would use the JFR API directly to capture `jdk.FileRead` events. The test would construct a JAR that previously triggered one read per entry, and assert that after this change, only a single read is needed to initialize a compiler whose classpath includes that JAR.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/24176#issuecomment-2749547902


More information about the compiler-dev mailing list