Skip to content

HDDS-16716. Add description for ozone.scm.ec.pipeline.per.volume.factor - #11415

Merged
adoroszlai merged 1 commit into
apache:masterfrom
lpavanvenkat:HDDS-16716
Oct 6, 2026
Merged

adoroszlai merged 1 commit into
apache:masterfrom
lpavanvenkat:HDDS-16716

Conversation

@lpavanvenkat

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

The SCM configuration ozone.scm.ec.pipeline.per.volume.factor had TODO as its description, so the generated ozone-default.xml and the config docs did not explain what the property does.

This PR replaces the placeholder with a description based on how the factor is used in WritableECContainerProvider#getMaximumPipelines. For each EC replication config, SCM limits the number of open pipelines to:

max(factor * total healthy volumes / (data + parity), ozone.scm.ec.pipeline.minimum)

Setting the factor to 0 disables the volume-based limit, so only ozone.scm.ec.pipeline.minimum applies.

This is a description-only change; there is no change in behavior.

What is the link to the Apache JIRA

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

How was this patch tested?

  • Ran ./hadoop-ozone/dev-support/checks/checkstyle.sh (0 violations).
  • Rebuilt hdds-server-scm and verified that the generated hdds-server-scm-default.xml contains the new description for ozone.scm.ec.pipeline.per.volume.factor.

@lpavanvenkat lpavanvenkat changed the title Add description for ozone.scm.ec.pipeline.per.volume.factor HDDS-16716. Add description for ozone.scm.ec.pipeline.per.volume.factor Oct 6, 2026
@github-actions github-actions Bot added the scm label Oct 6, 2026

@sravani-revuri sravani-revuri 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.

LGTM.

@adoroszlai
adoroszlai merged commit a99fabb into apache:master Oct 6, 2026
44 of 46 checks passed
@adoroszlai

Copy link
Copy Markdown
Contributor

Thanks @lpavanvenkat for the patch, @aryangupta1998, @sravani-revuri for the review.

errose28 added a commit that referenced this pull request Oct 6, 2026
* master: (64 commits)
  HDDS-16716. Add description for ozone.scm.ec.pipeline.per.volume.factor (#11415)
  HDDS-16008. PutBlocks from Flushes also go without Raft (#11356)
  HDDS-16666. Flush SCM transaction in memory during apply transaction (#11409)
  HDDS-15749. Run specific JUnit tests if possible (#10671)
  HDDS-16362. GetObjectAttributes ObjectParts should return Part entries for FSO buckets (#11242).
  HDDS-16643. Remove CleanupTableInfo mechanism (#11365)
  HDDS-16721. StreamBlockInputStream.read() returns a negative value for bytes 0x80 to 0xFF (#11411)
  HDDS-16241. gRPC deadline kills long-lived block streams after 30 seconds and the client never recovers (#11080)
  HDDS-15991. Speed up deleted table scans in quota repair (#11386)
  HDDS-16674. Bump awssdk to 2.55.6 (#11407)
  HDDS-16673. Avoid redundant ListBuckets RPCs when S3 bucket listing reaches the end (#11397)
  HDDS-16708. Let dependabot ignore iceberg minor version upgrades (#11398)
  HDDS-16713. Bump develocity-maven-extension to 2.6.0 (#11405)
  HDDS-16300. Allow OM to dynamically reconfigure its SCM node list without a restart (#11218)
  HDDS-15089. Support S3 per request read consistency (#11252)
  HDDS-16704. ReadBlock fails with IllegalStateException when a response is shorter than responseDataSize (#11402)
  HDDS-16631. Fix chooseRandom for rack names with common prefixes (#11401)
  HDDS-16654. Replace usage of deprecated finalize() in OM (#11376)
  HDDS-16658. Reuse source key details when opening input stream in S3 CopyObject (#11396)
  HDDS-16711. Bump moment to 2.31.0 (#11373)
  ...

Conflicts:
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/node/HealthyReadOnlyNodeHandler.java
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/node/NodeStateManager.java
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/node/SCMNodeManager.java
hadoop-hdds/server-scm/src/test/java/org/apache/hadoop/hdds/scm/ha/TestSCMStateMachine.java
hadoop-hdds/server-scm/src/test/java/org/apache/hadoop/hdds/scm/node/TestDeadNodeHandler.java
hadoop-hdds/server-scm/src/test/java/org/apache/hadoop/hdds/scm/node/TestNodeStateManager.java
hadoop-ozone/client/src/test/java/org/apache/hadoop/ozone/client/rpc/TestRpcClient.java
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/OmUtils.java
hadoop-ozone/common/src/test/java/org/apache/hadoop/ozone/om/ha/TestHadoopRpcOMFollowerReadFailoverProxyProvider.java
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/shell/TestOzoneShellHA.java
hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/OzoneManagerStateMachine.java
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/response/upgrade/OMCancelPrepareResponse.java
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/response/upgrade/OMCompleteFinalizeUpgradeResponse.java
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/response/upgrade/OMPrepareResponse.java
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/protocolPB/OzoneManagerRequestHandler.java
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/ratis/TestOzoneManagerStateMachine.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants