Skip to content

HDDS-15835. Count incomplete MPU parts in quota repair#10794

Closed
chihsuan wants to merge 1 commit into
apache:masterfrom
chihsuan:master
Closed

HDDS-15835. Count incomplete MPU parts in quota repair#10794
chihsuan wants to merge 1 commit into
apache:masterfrom
chihsuan:master

Conversation

@chihsuan

@chihsuan chihsuan commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This change updates QuotaRepairTask to include committed parts from incomplete multipart uploads when rebuilding the bucket usedBytes.

Previously, quota repair omitted these parts. A subsequent multipart upload abort released their replicated size, which could cause usedBytes to become negative. The repair task now scans the multipartInfoTable from its checkpoint. It calculates legacy MPU parts using the same replicated-size calculation as abort and uses OMMultipartUploadUtils for split-part MPUs. It adds only usedBytes; MPU parts do not consume usedNamespace.

The change also adds OBS and FSO coverage for legacy and split-parts MPU entries, part replacement, namespace accounting, and aborting after repair.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-15835

How was this patch tested?

  • mvn -pl :ozone-manager test -Dtest=TestQuotaRepairTask -DskipShade -DskipRecon -DskipDocs
  • ./hadoop-ozone/dev-support/checks/checkstyle.sh

Generated-by: Codex (GPT-5)

Copilot AI review requested due to automatic review settings July 18, 2026 03:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chihsuan
chihsuan marked this pull request as ready for review July 18, 2026 03:30
@ivandika3
ivandika3 requested a review from sumitagrawl July 18, 2026 03:31
@chihsuan

Copy link
Copy Markdown
Contributor Author

My apologies. I accidentally opened this PR from my fork’s master branch. I’ll close it and recreate the same change from the dedicated HDDS-15835 branch.

@chihsuan chihsuan closed this Jul 18, 2026
@chihsuan

Copy link
Copy Markdown
Contributor Author

See #10795 🙏

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