gh-yzou commented on code in PR #4237:
URL: https://github.com/apache/polaris/pull/4237#discussion_r3314921888


##########
runtime/service/src/main/java/org/apache/polaris/service/catalog/common/CatalogUtils.java:
##########
@@ -95,4 +143,177 @@ public static void validateLocationsForTableLike(
               }
             });
   }
+
+  /**
+   * Validate no location overlap exists between the entity path and its 
sibling entities. This
+   * resolves all siblings at the same level as the target entity (namespaces 
if the target entity
+   * is a namespace whose parent is the catalog, namespaces and tables 
otherwise) and checks the
+   * base-location property of each. The target entity's base location may not 
be a prefix or a
+   * suffix of any sibling entity's base location.
+   */
+  public static <T extends PolarisEntity & LocationBasedEntity> void 
validateNoLocationOverlap(

Review Comment:
   those functions are pure copy from IcebergCatalog.java



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to