-
Notifications
You must be signed in to change notification settings - Fork 588
HDDS-13571. Add upgrade action for NSSummary aggregated totals Improvement. #8938
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@devmadhuu @sumitagrawl Please take a look |
devmadhuu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ArafatKhan2198 for the patch. Changes LGTM +1
Tested with a cluster and also attached the Log with ExplanationAs shown in the log below, the NSSummaryTask was triggered by the upgrade handler, but the tree was not built immediately because the required tables and NSSummary tasks had not yet been registered. The registration and initialization steps must happen first. Only after those are complete can the NSSummaryTask run successfully. You can see in the sequence below:
|
sumitagrawl
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ArafatKhan2198 Given few comments
...on/src/main/java/org/apache/hadoop/ozone/recon/upgrade/NSSummaryAggregatedTotalsUpgrade.java
Outdated
Show resolved
Hide resolved
hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/ReconUtils.java
Outdated
Show resolved
Hide resolved
.../recon-codegen/src/main/java/org/apache/ozone/recon/schema/SchemaVersionTableDefinition.java
Outdated
Show resolved
Hide resolved
|
Thanks for the comments @sumitagrawl can you please take a look again - The upgrade action works properly now - |
devmadhuu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ArafatKhan2198 for the patch. LGTM +1
sumitagrawl
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What changes were proposed in this pull request?
Add upgrade action that triggers asynchronous NSSummary tree rebuild during Recon layout feature finalization. This ensures the NSSummary tree is rebuilt when upgrading to support materialized totals, without blocking Recon startup.
What this PR includes:
NSSummaryAggregatedTotalsUpgradeimplementingReconUpgradeActionand annotated to run at FINALIZE.NSSUMMARY_AGGREGATED_TOTALSentry toReconLayoutFeature(feature version 3) and registration of the upgrade action via the annotation-based scanner.ReconUtils.triggerAsyncNSSummaryRebuild(...)invoked by the upgrade action to schedule the rebuild without blocking startup.NSSummaryTask's unified rebuild state check.omMetadataManager.isOmTablesInitialized()before running the rebuild, with graceful timeout and interruption handling.NSSummaryTaskWithFSO.reprocessWithFSO(...)to skip processing when OM tables (e.g., directory table) are not yet initialized to avoid NPEs.What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-13571
How was this patch tested?
Tested manually