amogh-jahagirdar commented on PR #11165: URL: https://github.com/apache/iceberg/pull/11165#issuecomment-2366931136
Thanks @singhpk234! At the moment the [catalog API for renameTable](https://github.com/apache/iceberg/blob/main/api/src/main/java/org/apache/iceberg/catalog/Catalog.java#L315) does not specify what the expected exceptions are for the 403 or throttling but I agree that we may as well make those Iceberg exceptions in the catalog implementation. I wouldn't be opposed to upleveling the exception handling logic in `GlueTableOperations` to `GlueToToIcebergConverter` and then using that logic where relevant. -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
