RFR: 8259009: G1 heap summary should be shown in "Heap Parameters" window on HSDB [v2]

Yasumasa Suenaga ysuenaga at openjdk.java.net
Fri Jan 22 10:46:55 UTC 2021


On Fri, 22 Jan 2021 08:23:17 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> 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.

@tschatzl Thanks for the comment! I fixed it.

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

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


More information about the serviceability-dev mailing list