pvary commented on code in PR #8918:
URL: https://github.com/apache/iceberg/pull/8918#discussion_r1383401740


##########
hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java:
##########
@@ -261,6 +261,12 @@ public void renameTable(TableIdentifier from, 
TableIdentifier originalTo) {
     } catch (InterruptedException e) {
       Thread.currentThread().interrupt();
       throw new RuntimeException("Interrupted in call to rename", e);
+    } catch (RuntimeException e) {

Review Comment:
   Taking a quick look, I do not like this change.
   
   Why is this `RuntimeException`?
   
   This code tries to capture this case:
   ```
       } catch (AlreadyExistsException e) {
         throw new org.apache.iceberg.exceptions.AlreadyExistsException(
             "Table already exists: %s", to);
       }
   ```
   
   Do we have checks for the other cases as well?
   My first guess would be that `MetastoreUtil.alterTable` messes up the Hive 
exceptions, and maybe all of them are off?



-- 
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

Reply via email to