On Fri, 23 Jul 2021 14:58:01 GMT, Jaikiran Pai wrote:
>> Can I please get a review for this proposed fix for the issue reported in
>> https://bugs.openjdk.java.net/browse/JDK-8190753?
>>
>> The commit here checks for the size of the zip entry before trying to create
>> a
On Fri, 23 Jul 2021 16:22:01 GMT, Lance Andersen wrote:
> > Thank you for the review Alan.
> > @LanceAndersen, I've run the tier1 tests locally with the latest PR and
> > they have passed without any regressions. Given that we changed the
> > implementation to wrap ByteArrayOutputStream
On Fri, 23 Jul 2021 15:23:07 GMT, Jaikiran Pai wrote:
> Thank you for the review Alan.
>
> @LanceAndersen, I've run the tier1 tests locally with the latest PR and they
> have passed without any regressions. Given that we changed the implementation
> to wrap ByteArrayOutputStream instead of
On Fri, 23 Jul 2021 14:58:01 GMT, Jaikiran Pai wrote:
>> Can I please get a review for this proposed fix for the issue reported in
>> https://bugs.openjdk.java.net/browse/JDK-8190753?
>>
>> The commit here checks for the size of the zip entry before trying to create
>> a
> Can I please get a review for this proposed fix for the issue reported in
> https://bugs.openjdk.java.net/browse/JDK-8190753?
>
> The commit here checks for the size of the zip entry before trying to create
> a `ByteArrayOutputStream` for that entry's content. A new jtreg test has been
>