Skip to content

Support collect ZGC memory pool metrics#622

Merged
wu-sheng merged 4 commits intoapache:mainfrom
Z-Beatles:feature/20231013_zgc_support
Oct 20, 2023
Merged

Support collect ZGC memory pool metrics#622
wu-sheng merged 4 commits intoapache:mainfrom
Z-Beatles:feature/20231013_zgc_support

Conversation

@Z-Beatles
Copy link
Contributor

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

@wu-sheng
Copy link
Member

So, this relies on the protocol update.
Java 9.1 can't release before OAP 9.7, right?

And do we expect UI setting change?

@wu-sheng wu-sheng added the enhancement New feature or request label Oct 13, 2023
type = PoolType.METASPACE_USAGE;
} else if (name.equals("Compressed Class Space")) {
type = PoolType.COMPRESSED_CLASS_SPACE_USAGE;
} else if (name.equals("CodeHeap 'non-nmethods'")) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the single quote from official definition? It is strange to read.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, as we can see i added a panel JVM Memory Detail (MB) on UI to show memory pool detail after JVM Memory (MB).
image

@Z-Beatles
Copy link
Contributor Author

Z-Beatles commented Oct 16, 2023

So, this relies on the protocol update. Java 9.1 can't release before OAP 9.7, right?

And do we expect UI setting change?

yes, i will pull a new request later on main repo for UI setting change.

@wu-sheng wu-sheng added this to the 9.1.0 milestone Oct 20, 2023
wu-sheng
wu-sheng previously approved these changes Oct 20, 2023
@wu-sheng wu-sheng merged commit c15ac30 into apache:main Oct 20, 2023
@wu-sheng wu-sheng mentioned this pull request Oct 20, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants