apache / iceberg

Apache Iceberg
https://iceberg.apache.org/
Apache License 2.0
6.19k stars 2.16k forks source link

Table rename in Glue Catalog throws Incorrect `AlreadyExistsException` #11155

Open ZygimantasKoncius opened 3 days ago

ZygimantasKoncius commented 3 days ago

Apache Iceberg version

1.6.1 (latest release)

Query engine

None

Please describe the bug 🐞

Based on the Catalog interface renameTable should throw org.apache.iceberg.exceptions.AlreadyExistsException, but instead it throws software.amazon.awssdk.services.glue.model.AlreadyExistsException

Bug is that: https://github.com/apache/iceberg/blob/06ed235f9748465a8d85f8772f052bfe64b2f955/aws/src/main/java/org/apache/iceberg/aws/glue/GlueCatalog.java#L419 isn't wrapped in software.amazon.awssdk.services.glue.model.AlreadyExistsException e and rethrown with Iceberg's AlreadyExistsException.

Willingness to contribute

jishangarg commented 3 days ago

Hi Team, Can somebody please review my PR #11165 ?