7042: Bring back summary screen for Statistics on Object Allocation outside TLAB.

[Suchitainf]

This PR addresses the enhancement of adding General screen (providing summary) and By Class (table and graph) for allocations inside and outside TLAB.

The changes include the enhancement "7043: Bring back summary screen for Statistics on Object Allocation inside TLAB" as well.

Note: This is bring back feature i.e. which was present in JMC 5.5 but was missing in recent releases.

Please review the change and provide your valuable comments.

---

Progress

• Commit message must refer to an issue
• Change must be properly reviewed

Issue

• JMC-7042: Bring back summary screen for Statistics on Object Allocation outside TLAB.

Reviewers

• Alex Macdonald (@aptmac - Committer)
• Marcus Hirt (@thegreystone - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jmc pull/269/head:pull/269
$ git checkout pull/269

Update a local copy of the PR:
$ git checkout pull/269
$ git pull https://git.openjdk.java.net/jmc pull/269/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 269

View PR using the GUI difftool:
$ git pr show -t 269

Using diff file

Download this PR as a diff file:
https://git.openjdk.java.net/jmc/pull/269.diff

[bridgekeeper]

👋 Welcome back schaturvedi! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[mlbridge]

Webrevs

• 02: Full - Incremental (5a9f8bff)
• 01: Full - Incremental (9c1d3a4e)
• 00: Full (225121d2)

[aptmac]

Overall the page looks good, I've left a couple of review comments inline.

For anyone that doesn't feel like building this pr, here's a screenshot of what it looks like:
Tab group:

Class Page:

Summary Page:

[openjdk]

@Suchitainf This change now passes all automated pre-integration checks.

ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.

After integration, the commit message for the final commit will be:

7042: Bring back summary screen for Statistics on Object Allocation outside TLAB.

Reviewed-by: aptmac, hirt

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 11 new commits pushed to the master branch:

• e838a74: 7304: JMC 8.0.1 fix formatting failure
• 66685ca: 7188: JMC fails to dump file and gets stuck when flightrecording is attempted on jmxremote connection
• ef101c2: 7172: fix spell mistake in secure store class
• a3fa120: 6920: UI improvements Reviewed-by: cries, bbanathur, schaturvedi
• 5f23f54: 6336: Remove Triple DES Cipher in Secure store
• ee428c5: 6398: Better JNDI Usage
• e34323d: 6141: JMC logs warnings when creating recordings
• 7f38790: 6967: Add Percentage Column w.r.t 'Profiling Samples' in Thread table
• 6723a70: 7299: Accessibility Issue - Too low contrast on rule result values
• 8bc1247: 7272: Method profiling page should show the frame type
• ... and 1 more: https://git.openjdk.java.net/jmc/compare/8e2988edfd081dc34770ef54c5657a6d072399a9...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

[aptmac]

/reviewers 2

[openjdk]

@aptmac Only the author of the pull request or Reviewers are allowed to change the number of required reviewers.

[Suchitainf]

/integrate

[openjdk]

Going to push as commit 81344ce.
Since your change was applied there have been 11 commits pushed to the master branch:

• e838a74: 7304: JMC 8.0.1 fix formatting failure
• 66685ca: 7188: JMC fails to dump file and gets stuck when flightrecording is attempted on jmxremote connection
• ef101c2: 7172: fix spell mistake in secure store class
• a3fa120: 6920: UI improvements Reviewed-by: cries, bbanathur, schaturvedi
• 5f23f54: 6336: Remove Triple DES Cipher in Secure store
• ee428c5: 6398: Better JNDI Usage
• e34323d: 6141: JMC logs warnings when creating recordings
• 7f38790: 6967: Add Percentage Column w.r.t 'Profiling Samples' in Thread table
• 6723a70: 7299: Accessibility Issue - Too low contrast on rule result values
• 8bc1247: 7272: Method profiling page should show the frame type
• ... and 1 more: https://git.openjdk.java.net/jmc/compare/8e2988edfd081dc34770ef54c5657a6d072399a9...master

Your commit was automatically rebased without conflicts.

[openjdk]

@Suchitainf Pushed as commit 81344ce.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.