Skip to content

HDDS-14555. [Recon] Clarify Open Key Bytes breakdown in Cluster Capacity page#9705

Draft
priyeshkaratha wants to merge 1 commit intoapache:masterfrom
priyeshkaratha:HDDS-14555
Draft

HDDS-14555. [Recon] Clarify Open Key Bytes breakdown in Cluster Capacity page#9705
priyeshkaratha wants to merge 1 commit intoapache:masterfrom
priyeshkaratha:HDDS-14555

Conversation

@priyeshkaratha
Copy link
Contributor

What changes were proposed in this pull request?

This pull request enhances the Apache Ozone Recon UI by providing a more detailed breakdown of "Open Key Bytes" on the Cluster Capacity page. Previously, only a single aggregated value was displayed. The changes introduce a new data structure in the backend to differentiate between "Open Key/File Bytes" and "Multipart Key Bytes," which is then exposed via the Recon API and presented to users in an informative tooltip on the frontend. This improvement offers greater transparency and clarity regarding storage utilization related to open keys.

What is the link to the Apache JIRA

HDDS-14555

How was this patch tested?

Tested UI manually.

@priyeshkaratha priyeshkaratha changed the title HDDS-14555. Clarify Open Key Bytes breakdown in Cluster Capacity page HDDS-14555. [Recon] Clarify Open Key Bytes breakdown in Cluster Capacity page Feb 4, 2026
Comment on lines +65 to +69
onSuccess: (data) => {
console.log('Storage Distribution API Response:', data);
console.log('usedSpaceBreakdown:', data.usedSpaceBreakdown);
console.log('openKeyBytes:', data.usedSpaceBreakdown?.openKeyBytes);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the console.log() required here?
I think we can remove it if it was used for testing out changes

Copy link
Contributor

@devabhishekpal devabhishekpal left a comment

Choose a reason for hiding this comment

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

Thanks for the patch @priyeshkaratha.
Overall the UI changes looks good to me, one minor comment, please take a look.

Could you also attach the UI screenshots in the PR description?
An also update the test description that the java patch was tested with Unit tests? since the tests got modified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants