On Sun, 12 Mar 2023 11:13:51 GMT, Lance Andersen <lan...@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

Reply via email to