fix parentChildEmptyNullConfig

Netflix · Oct 28, 2024 · 0be8524 · 0be8524
1 parent 1aada77
commit 0be8524
Showing 1 changed file with 6 additions and 12 deletions.
diff --git a/.../java/com/netflix/metacat/common/server/properties/ParentChildRelationshipProperties.java b/.../java/com/netflix/metacat/common/server/properties/ParentChildRelationshipProperties.java
@@ -58,11 +58,9 @@ public ParentChildRelationshipProperties(@Nullable final Environment env) {
      * @param  configStr configString
      */
     public void setMaxAllowPerTablePerRelType(@Nullable final String configStr) {
-        if (configStr == null || configStr.isEmpty()) {
-            return;
-        }
         try {
-            this.maxAllowPerTablePerRelType = parseNestedConfigString(configStr);
+            this.maxAllowPerTablePerRelType = configStr == null || configStr.isEmpty()
+                ? new HashMap<>() : parseNestedConfigString(configStr);
         } catch (Exception e) {
             log.error("Fail to apply configStr = {} for maxAllowPerTablePerRelType", configStr, e);
         }
@@ -74,11 +72,9 @@ public void setMaxAllowPerTablePerRelType(@Nullable final String configStr) {
      * @param  configStr configString
      */
     public void setMaxAllowPerDBPerRelType(@Nullable final String configStr) {
-        if (configStr == null || configStr.isEmpty()) {
-            return;
-        }
         try {
-            this.maxAllowPerDBPerRelType = parseNestedConfigString(configStr);
+            this.maxAllowPerDBPerRelType = configStr == null || configStr.isEmpty()
+                ? new HashMap<>() : parseNestedConfigString(configStr);
         } catch (Exception e) {
             log.error("Fail to apply configStr = {} for maxCloneAllowPerDBPerRelType", configStr);
         }
@@ -89,11 +85,9 @@ public void setMaxAllowPerDBPerRelType(@Nullable final String configStr) {
      * @param  configStr configString
      */
     public void setDefaultMaxAllowPerRelType(@Nullable final String configStr) {
-        if (configStr == null || configStr.isEmpty()) {
-            return;
-        }
         try {
-            this.defaultMaxAllowPerRelType =
+            this.defaultMaxAllowPerRelType = configStr == null || configStr.isEmpty() ?
+                new HashMap<>() :
                 Arrays.stream(configStr.split(";"))
                     .map(entry -> entry.split(","))
                     .collect(Collectors.toMap(