On Mon, 11 Oct 2021 00:38:22 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> This PR fix the issue of JDK-8274620 
>> (resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java is timing out)
>> The root cause is that `BufferedOutputStream` should be used when writting 
>> data to file.
>
> LGTM.
> 
> This code should be rewritten to use try-with-resources at some point.
> 
> Thanks,
> David

Thanks @dholmes-ora for review and comments! I will create another issue in JBS 
for the try-with-resources change.

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

PR: https://git.openjdk.java.net/jdk/pull/5860

Reply via email to