ajantha-bhat commented on PR #6887: URL: https://github.com/apache/iceberg/pull/6887#issuecomment-1449489814
> How critical is it to have a special RuntimeException class to indicate that something went wrong, or could we make things work by directly using RuntimeException? I think Iceberg follows a style of wrapping the `RuntimeException` with a class to indicate that Iceberg is aware of this exception. So, I followed the same style. In the original PR #4776, @rdblue also suggested using some other exception than `IOException`. So, I assumed it is `RuntimeException` and wrapped it. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
