Skip to content

HBASE-29940 GC Collector stats can't be displayed properly in JDK21#7822

Merged
Apache9 merged 1 commit intoapache:masterfrom
liuxiaocs7:HBASE-29940
Mar 6, 2026
Merged

HBASE-29940 GC Collector stats can't be displayed properly in JDK21#7822
Apache9 merged 1 commit intoapache:masterfrom
liuxiaocs7:HBASE-29940

Conversation

@liuxiaocs7
Copy link
Contributor

@liuxiaocs7 liuxiaocs7 changed the title HBASE-29940 GC Collector stats can't be display properly in JDK21 HBASE-29940 GC Collector stats can't be displayed properly in JDK21 Feb 28, 2026
@liuxiaocs7
Copy link
Contributor Author

CC: @Apache9 and @PDavid, please help take a look when free, thanks!

Copy link
Contributor

@Apache9 Apache9 left a comment

Choose a reason for hiding this comment

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

Could you please explain a bit on what is the real fix here?

Thanks.

@liuxiaocs7
Copy link
Contributor Author

Could you please explain a bit on what is the real fix here?

Thanks.

Thanks for the comments, In short, the default implementation uses two GC Collectors, but in higher JDK versions, the situation changes, it could be two, three, or four. Therefore, we need a more universal approach to accommodate this scenario.

For more details and images, attached in HBASE-29940

Comment on lines +110 to +111
<% idx = 0; long totalGcTime = 0; for (GarbageCollectorMXBean gc : gcBeans) { totalGcTime += gc.getCollectionTime(); %>
<div class="tab-pane <%= idx == 0 ? "active" : "" %>" id="tab_gc_<%= idx %>" role="tabpanel">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the fix

@Apache9 Apache9 merged commit 54123b3 into apache:master Mar 6, 2026
4 of 7 checks passed
Apache9 pushed a commit that referenced this pull request Mar 6, 2026
…7822)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 54123b3)
Apache9 pushed a commit that referenced this pull request Mar 6, 2026
…7822)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 54123b3)
Apache9 pushed a commit that referenced this pull request Mar 6, 2026
…7822)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 54123b3)
Apache9 pushed a commit that referenced this pull request Mar 6, 2026
…7822)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 54123b3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants