Skip to content

HDDS-13575. Update Erasure Coding section in the Topology Awareness user doc#8942

Merged
jojochuang merged 2 commits intoapache:masterfrom
jojochuang:HDDS-13575
Sep 19, 2025
Merged

HDDS-13575. Update Erasure Coding section in the Topology Awareness user doc#8942
jojochuang merged 2 commits intoapache:masterfrom
jojochuang:HDDS-13575

Conversation

@jojochuang
Copy link
Contributor

What changes were proposed in this pull request?

HDDS-13575. Update Erasure Coding section in the Topology Awareness user doc

Please describe your PR in detail:

  • Add EC section in the topology awareness user doc
  • Generated-by: Google Gemini 2.5 Pro + Gemini Cli

What is the link to the Apache JIRA

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

How was this patch tested?

@jojochuang jojochuang added documentation Improvements or additions to documentation AI-gen labels Aug 14, 2025
@sodonnel
Copy link
Contributor

This is probably going to conflct with #8947 so maybe we should complete it and then review / rebase this one?

…ser doc.

Change-Id: I66c09d82a3eb89edff3538c59fd53e131065cab8

Revert "HDDS-13575. Update Erasure Coding section in the Topology Awareness user doc."

This reverts commit 515a73b.

HDDS-13575. Docs: Add EC placement policy to topology doc.

This commit adds a new section to the topology documentation to explain the Erasure Coded container placement policy (`SCMContainerPlacementRackScatter`).

The new section includes:
- The default policy for EC containers.
- How the policy works to scatter blocks across racks.
- The configuration key to customize the policy.
- The limitations of the policy.

This commit also corrects the reference numbers in the document.

Change-Id: Iab13978463b6266fb9395754954ba5c12e878f3e
@jojochuang
Copy link
Contributor Author

Rebased

@jojochuang jojochuang marked this pull request as ready for review September 16, 2025 05:15
@jojochuang jojochuang requested a review from sodonnel September 16, 2025 05:15
Copy link
Contributor

@sodonnel sodonnel left a comment

Choose a reason for hiding this comment

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

I left one comment, the rest looks good.

Change-Id: Ifadfea1bcf16fabfd56e1e3c6e4b81bfa864bb96
@jojochuang jojochuang merged commit 7cf9f65 into apache:master Sep 19, 2025
3 checks passed
@jojochuang
Copy link
Contributor Author

Thanks @sodonnel

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

Labels

AI-gen documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants