diff --git a/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/client/ReplicationConfigValidator.java b/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/client/ReplicationConfigValidator.java index 7090c76563b..36d95251ed9 100644 --- a/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/client/ReplicationConfigValidator.java +++ b/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/client/ReplicationConfigValidator.java @@ -61,9 +61,11 @@ public ReplicationConfig validate(ReplicationConfig replicationConfig) { replication = ecConfig.getCodec() + "-" + ecConfig.getData() + "-" + ecConfig.getParity() + "-{CHUNK_SIZE}"; } - throw new IllegalArgumentException("Invalid replication config " + - "for type " + replicationConfig.getReplicationType() + - " and replication " + replication); + throw new IllegalArgumentException( + "Invalid data-parity replication config " + + "for type " + replicationConfig.getReplicationType() + + " and replication " + replication + "." + + " Supported data-parity are 3-2,6-3,10-4"); } } return replicationConfig; diff --git a/hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/client/TestReplicationConfigValidator.java b/hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/client/TestReplicationConfigValidator.java index bbdc78c94ab..acf657b4ba3 100644 --- a/hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/client/TestReplicationConfigValidator.java +++ b/hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/client/TestReplicationConfigValidator.java @@ -51,8 +51,10 @@ public void testValidation() { try { validator.validate(new ECReplicationConfig(invalidEcConfig1)); } catch (IllegalArgumentException ex) { - GenericTestUtils.assertExceptionContains("Invalid replication " + - "config for type EC and replication xor-6-4-{CHUNK_SIZE}", ex); + GenericTestUtils.assertExceptionContains( + "Invalid data-parity replication " + + "config for type EC and replication xor-6-4-{CHUNK_SIZE}. " + + "Supported data-parity are 3-2,6-3,10-4", ex); } }