eric-maynard commented on code in PR #1298: URL: https://github.com/apache/polaris/pull/1298#discussion_r2028094941
########## service/common/src/main/java/org/apache/polaris/service/catalog/common/CatalogHandler.java: ########## @@ -356,4 +348,23 @@ protected void authorizeRenameTableLikeOperationOrThrow( initializeCatalog(); } + + /** + * Helper function for when a TABLE_LIKE entity is not found & we want to throw the appropriate + * exception + * + * @param subType The subtype of the entity that the exception should report doesn't exist + */ + public static void throwNotFoundException( Review Comment: That kind of makes it sound like it's throwing an exception called `TableLikeEntityNotFoundException`, no? I'm mostly thinking about the catalog APIs here, we are throwing Iceberg exceptions like `NoSuchTableException` -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org