RFR: 8304020: Speed up test/jdk/java/util/zip/ZipFile/TestTooManyEntries.java and clarify its purpose [v3]

Eirik Bjorsnos duke at openjdk.org
Sun Mar 12 11:24:17 UTC 2023


On Sun, 12 Mar 2023 11:13:51 GMT, Lance Andersen <lancea at openjdk.org> wrote:

> I think the changes look good overall. Thank you for this. I am not sure that the `@requires` is needed at this point.

I think it is still needed. ZipOutputStream needs to keep all entries in memory until it can write the CEN entries. I tried reducing it to 7GB, but that failed with OutOfMemoryError.

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

PR: https://git.openjdk.org/jdk/pull/12991


More information about the core-libs-dev mailing list