[jdk11u-dev] Integrated: 8237354: Add option to jcmd to write a gzipped heap dump

[Bin Liao]

On Mon, 9 Aug 2021 07:08:36 GMT, Bin Liao <github.com+3094961+buddyliao at openjdk.org> wrote:

> Backport of the heap dump enhancement: Add option to jcmd to write a gzipped heap dump
> 
> Patch did not apply cleanly, I had to resolve/modify
> 
> 1) src/hotspot/share/services/heapDumper.cpp
> function "int dump(const char* path, bool overwrite = false)" has some different from the jdk-master, since patch 7cbb67a3f8adc83a5b51c092a66480d7b22a6bea is backport before this one, so this patch can't backport clearly
> 
> 2) src/hotspot/share/services/heapDumperCompression.cpp
> MonitorLockerEx is named MonitorLocker in jdk-master, MonitorLockerEx.wait should have parameters, the constructor of  MonitorLockerEx and MutexUnlockerEx also different from jdk-master
> 
> 3) src/hotspot/share/services/heapDumperCompression.hpp
> FileWriter is different from jdk-master since backport of 7cbb67a3f8adc83a5b51c092a66480d7b22a6bea before this patch.
> 
> I have built it on linux x86_64 and run test-tier1

This pull request has now been integrated.

Changeset: 1868c15d
Author:    Bin Liao <buddyliao at tencent.com>
Committer: Christoph Langer <clanger at openjdk.org>
URL:       https://git.openjdk.java.net/jdk11u-dev/commit/1868c15d4e5d03895a4a07edacd864021dfbac57
Stats:     1758 lines in 13 files changed: 1637 ins; 52 del; 69 mod

8237354: Add option to jcmd to write a gzipped heap dump

Reviewed-by: rschmelter, clanger
Backport-of: 19be49714356f2f598924c9fefa6358679ab6dbe

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

PR: https://git.openjdk.java.net/jdk11u-dev/pull/209