[PATCH] JMC-6082: JOverflow does not handle large heap dumps well

[Marcus Hirt]

LGTM!

Kind regards,
Marcus

On Fri, Jul 12, 2019 at 4:50 PM Arvin Kangcheng Xu <kxu at redhat.com> wrote:
>
> Hello all,
>
> This patch addresses JMC-6082 (JOverflow does not handle large
> heap dumps well) [0].
>
> The problem: the dump file is closed prematurely after only a part of
> the file content is pre-fetched into the cache. In case of cache
> misses, subsequent reads from an already-closed file will result in a
> stream closed exception.
>
> Please see the attached patch, and let me know your thoughts. Thank
> you very much!
>
> Regards,
> Kangcheng
>
> [0] https://bugs.openjdk.java.net/browse/JMC-6082