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