diff --git a/metacat-functional-tests/src/functionalTest/groovy/com/netflix/metacat/ParentChildRelMetadataServiceSpec.groovy b/metacat-functional-tests/src/functionalTest/groovy/com/netflix/metacat/ParentChildRelMetadataServiceSpec.groovy index d518a916..df1c46a8 100644 --- a/metacat-functional-tests/src/functionalTest/groovy/com/netflix/metacat/ParentChildRelMetadataServiceSpec.groovy +++ b/metacat-functional-tests/src/functionalTest/groovy/com/netflix/metacat/ParentChildRelMetadataServiceSpec.groovy @@ -604,10 +604,10 @@ class ParentChildRelMetadataServiceSpec extends Specification{ where: maxAllow | defaultMaxAllowPerRelStr | maxAllowPerDBPerRelTypeStr | maxAllowPerTablePerRelTypeStr | expectedChildAllowCount 3 | "" | "" | "" | 3 - 3 | "Other,5" | "CLONE,other,2" | "CLONE,testhive/test/other,2" | 3 + 3 | "Other,5" | "Other,other,2" | "Other,testhive/test/other,2" | 3 1 | "CLONE,5" | "" | "" | 5 1 | "CLONE,5;Other,3" | "" | "" | 5 - 1 | "CLONE,5;Other,3" | "CLONE,other,2" | "CLONE,testhive/test/other,2" | 5 + 1 | "CLONE,5;Other,3" | "Other,other,2" | "Other,testhive/test/other,2" | 5 1 | "CLONE,5" | "CLONE,test,3" | "" | 3 1 | "CLONE,5;Other,3" | "CLONE,test,3;CLONE,other,2"| "" | 3 1 | "CLONE,5;Other,3" | "CLONE,test,3;OTHER,other,2"| "CLONE,testhive/test/other,2" | 3 diff --git a/metacat-metadata-mysql/src/main/java/com/netflix/metacat/metadata/mysql/MySqlParentChildRelMetaDataService.java b/metacat-metadata-mysql/src/main/java/com/netflix/metacat/metadata/mysql/MySqlParentChildRelMetaDataService.java index 60d84232..f7553a72 100644 --- a/metacat-metadata-mysql/src/main/java/com/netflix/metacat/metadata/mysql/MySqlParentChildRelMetaDataService.java +++ b/metacat-metadata-mysql/src/main/java/com/netflix/metacat/metadata/mysql/MySqlParentChildRelMetaDataService.java @@ -140,13 +140,6 @@ private void validateMaxAllow(final QualifiedName parentName, maxAllow = props.getMaxAllow(); } -// if (parentName.toString().equals("testhive/testmaxallow/parent")) { -// final String errorMsg = String.format( -// "Parent table: %s allow to have %s child table", -// parentName, maxAllow); -// throw new ParentChildRelServiceException(errorMsg); -// } - // if maxAllow < 0, this means we can create as many child table under the parent if (maxAllow < 0) { return;