Skip to content

HDDS-13578. [Docs] Add pipeline placement policy to Topology Awareness doc#8947

Merged
jojochuang merged 3 commits intoapache:masterfrom
jojochuang:HDDS-13578
Sep 2, 2025
Merged

HDDS-13578. [Docs] Add pipeline placement policy to Topology Awareness doc#8947
jojochuang merged 3 commits intoapache:masterfrom
jojochuang:HDDS-13578

Conversation

@jojochuang
Copy link
Contributor

Change-Id: Idd660a3e4cef13332ac796635f30f2ee63735b6a

What changes were proposed in this pull request?

HDDS-13578. [Docs] Add pipeline placement policy to Topology Awareness doc

Please describe your PR in detail:

  • Generated-by: Google Gemini Cli 2.5 Pro + Gemini Cli.
  • The previous documentation was ambiguous about the different placement and selection policies, leading to confusion.

This change restructures the document to clearly distinguish between:

  • Pipeline Creation Policy (ozone.scm.pipeline.placement.impl)
  • Pipeline Selection/Load Balancing Policy (hdds.scm.pipeline.choose.policy.impl)
  • Closed Container Replication Policy (ozone.scm.container.placement.impl)

This aligns the documentation with the behavior discovered in the source code.

What is the link to the Apache JIRA

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

How was this patch tested?

…s doc

Clarify placement and selection policies

The previous documentation was ambiguous about the different
placement and selection policies, leading to confusion.

This change restructures the document to clearly distinguish between:
- Pipeline Creation Policy (`ozone.scm.pipeline.placement.impl`)
- Pipeline Selection/Load Balancing Policy (`hdds.scm.pipeline.choose.policy.impl`)
- Closed Container Replication Policy (`ozone.scm.container.placement.impl`)

This aligns the documentation with the behavior discovered in the source code.

Change-Id: Idd660a3e4cef13332ac796635f30f2ee63735b6a
@jojochuang jojochuang added documentation Improvements or additions to documentation AI-gen labels Aug 15, 2025
@jojochuang jojochuang requested a review from whbing August 15, 2025 16:16
Change-Id: If79beeaff5ef481d9fee525ecf107619f703fc97
Change-Id: I4189f1a961ea1c8fc4c95a982f01ac3bb3d28256
@jojochuang jojochuang marked this pull request as ready for review August 28, 2025 01:31
@jojochuang jojochuang requested a review from sodonnel August 28, 2025 01:31
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.

LGTM

@jojochuang jojochuang merged commit 3948ca0 into apache:master Sep 2, 2025
25 checks passed
@jojochuang
Copy link
Contributor Author

Thanks @sodonnel merged to refresh doc

peterxcli pushed a commit to peterxcli/ozone that referenced this pull request Sep 10, 2025
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