commit a1ba9eaa55bb795e165f3082da7d99d953c5688f
author DaveTeng0 <109315747+DaveTeng0@users.noreply.github.com> Sun Aug 21 12:51:48 2022 -0700
committer GitHub <noreply@github.com> Sun Aug 21 21:51:48 2022 +0200
tree 4e99c26ecb84f28c16d8e040b599801e4b90c47f
parent 9fc0f9801dad8ad6fd76a8a2363b1def98361dc5

HDDS-7108. Make ReplicationConfig validation error message type-specific (#3702)

hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/client/ReplicationConfigValidator.java

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 93f83d1..e690663 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
@@ -62,12 +62,17 @@
               (ECReplicationConfig) replicationConfig;
         replication =  ecConfig.getCodec() + "-" + ecConfig.getData() +
                 "-" + ecConfig.getParity() + "-{CHUNK_SIZE}";
-      }
-      throw new IllegalArgumentException(
+        //EC type checks data-parity
+        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");
+      }
+      //Non-EC type
+      throw new IllegalArgumentException("Invalid replication config " +
+              "for type " + replicationConfig.getReplicationType() +
+              " and replication " + replication);
     }
     return replicationConfig;
   }