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