RFR: 8259009: G1 heap summary should be shown in "Heap Parameters" window on HSDB [v2]
Thomas Schatzl
tschatzl at openjdk.java.net
Fri Jan 22 08:26:42 UTC 2021
On Fri, 22 Jan 2021 06:51:59 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:
>> G1 heap summary (G1 Heap, summaries for each spaces) is shown on console even though I chosen "Heap Parameters" menu on HSDB. It should be shown on "Heap Parameters" window on HSDB.
>
> Yasumasa Suenaga has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>
> - Update copyright year
> - Merge remote-tracking branch 'upstream/master' into JDK-8259009
> - G1 heap summary should be shown in "Heap Parameters" window on HSDB
I would prefer to make the order of parameters uniform in `printG1HeapSummary(G1CollectedHeap g1h, PrintStream tty) {` too. I.e. there does not seem to be a reason to have only this function have `tty` as second parameter.
-------------
Changes requested by tschatzl (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1911
More information about the hotspot-gc-dev
mailing list