This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new 08c0ab84ba HDDS-13075. Fix default value in description of container
placement policy configs (#8511)
08c0ab84ba is described below
commit 08c0ab84ba548e303ce7c50e4ed3540f51ff36a2
Author: sreejasahithi <[email protected]>
AuthorDate: Wed May 28 20:11:42 2025 +0530
HDDS-13075. Fix default value in description of container placement policy
configs (#8511)
---
hadoop-hdds/common/src/main/resources/ozone-default.xml | 4 ++--
.../placement/algorithms/ContainerPlacementPolicyFactory.java | 3 +--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/hadoop-hdds/common/src/main/resources/ozone-default.xml
b/hadoop-hdds/common/src/main/resources/ozone-default.xml
index ae81dd2cc2..c94e21129c 100644
--- a/hadoop-hdds/common/src/main/resources/ozone-default.xml
+++ b/hadoop-hdds/common/src/main/resources/ozone-default.xml
@@ -916,7 +916,7 @@
The full name of class which implements
org.apache.hadoop.hdds.scm.PlacementPolicy.
The class decides which datanode will be used to host the container
replica. If not set,
-
org.apache.hadoop.hdds.scm.container.placement.algorithms.SCMContainerPlacementRandom
will be used as default
+
org.apache.hadoop.hdds.scm.container.placement.algorithms.SCMContainerPlacementRackAware
will be used as default
value.
</description>
</property>
@@ -928,7 +928,7 @@
The full name of class which implements
org.apache.hadoop.hdds.scm.PlacementPolicy.
The class decides which datanode will be used to host the container
replica in EC mode. If not set,
-
org.apache.hadoop.hdds.scm.container.placement.algorithms.SCMContainerPlacementRandom
will be used as default
+
org.apache.hadoop.hdds.scm.container.placement.algorithms.SCMContainerPlacementRackScatter
will be used as default
value.
</description>
</property>
diff --git
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/placement/algorithms/ContainerPlacementPolicyFactory.java
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/placement/algorithms/ContainerPlacementPolicyFactory.java
index aef4163588..950da7d9c8 100644
---
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/placement/algorithms/ContainerPlacementPolicyFactory.java
+++
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/placement/algorithms/ContainerPlacementPolicyFactory.java
@@ -38,7 +38,7 @@ public final class ContainerPlacementPolicyFactory {
private static final Class<? extends PlacementPolicy>
OZONE_SCM_CONTAINER_PLACEMENT_IMPL_DEFAULT =
- SCMContainerPlacementRandom.class;
+ SCMContainerPlacementRackAware.class;
private static final Class<? extends PlacementPolicy>
OZONE_SCM_CONTAINER_PLACEMENT_EC_IMPL_DEFAULT =
SCMContainerPlacementRackScatter.class;
@@ -62,7 +62,6 @@ public static PlacementPolicy getECPolicy(
ConfigurationSource conf, final NodeManager nodeManager,
NetworkTopology clusterMap, final boolean fallback,
SCMContainerPlacementMetrics metrics) throws SCMException {
- // TODO: Change default placement policy for EC
final Class<? extends PlacementPolicy> placementClass = conf
.getClass(ScmConfigKeys.OZONE_SCM_CONTAINER_PLACEMENT_EC_IMPL_KEY,
OZONE_SCM_CONTAINER_PLACEMENT_EC_IMPL_DEFAULT,
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
